Skip to content

Conversation

@nathanthorpe
Copy link
Contributor

@nathanthorpe nathanthorpe commented Jul 31, 2025

CI-747

Add tools for testing the following

  • Preprocess scripting (sample sheet generation)
  • File name patterns and auto sample metadata extraction

Refactor PreprocessDataset class 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_running method 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

@nathanthorpe nathanthorpe marked this pull request as draft July 31, 2025 23:11
)

@property
def developer_helper(self):
Copy link
Contributor Author

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?

Copy link
Contributor

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

@nathanthorpe nathanthorpe marked this pull request as ready for review July 31, 2025 23:44
@nathanthorpe nathanthorpe requested a review from a team July 31, 2025 23:44
)

@property
def developer_helper(self):
Copy link
Contributor

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

@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 5, 2025

@nathanthorpe nathanthorpe merged commit ccdb606 into main Aug 5, 2025
9 checks passed
@nathanthorpe nathanthorpe deleted the developer-tools branch August 5, 2025 21:41
@nathanthorpe nathanthorpe changed the title Developer tools for SDK CI-747 - Developer tools for SDK Aug 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants