-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update API in Suitcase #62
base: master
Are you sure you want to change the base?
Conversation
@licode, since we create the data broker with |
Now we may have multiple databrokers, so it is better to define it explicitly as an input. But it is better to pass in db instead of db.mds in case we want to use other functions from db. |
Exactly! =) |
@licode Did this code get ported to the suitcase repo? |
right, this is the doc for a new PR in suitcase. I will do rebasing here. |
DOC: update suitcase pre-processing docs
What's the status of this? Is this still relevant given our new plans for suitcase, @licode? |
Some thoughts on how to do preprocessing on the data. Thanks for suggestion from @tacaswell