-
Notifications
You must be signed in to change notification settings - Fork 1
CI-747 - Developer tools for SDK #164
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
Conversation
| ) | ||
|
|
||
| @property | ||
| def developer_helper(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
expose this on the portal root?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that's fine
| ) | ||
|
|
||
| @property | ||
| def developer_helper(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that's fine
…g out indexed reads
|



CI-747
Add tools for testing the following
Refactor
PreprocessDatasetclass to be more friendly around using it outside of the analysis runtime and add comments on the various attributes.Init preprocess logger in the
from_runningmethod to avoid multiple instances.Replace dataset params/info attributes with the raw dict, rather than the wrapper class provided by the openapi client.
See example notebook https://github.com/CirroBio/Cirro-SDK-Python/blob/developer-tools/samples/Integrating_pipelines.ipynb