Release 0.13.0 #180
Merged
Release 0.13.0 #180
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
add missing .
- set up pipestatmanager based on pipelinemanager init args - add pipestat argument group
might be used to discriminate between runs that did not execute any commands and runs that did
|
@stolarczyk could you write a few words about the state where you left the pypiper upgrade to pipestat? I don't have any notes written down about where you were in this process. I notice that many tests are failing and I am wondering if this was something you were in the middle of, or you though you had finished this and had it working at one point with the tests, and there was something else that broke these later on, or what. |
* Work towards pypier utilizing pipestat changes. Related: pepkit/pipestat#21 * Refactor to align with pipestat refactoring. Create default output schema to satisfy pipestat requirements. * Call pipestat backend for setting status. Set status for pipeline.manager. * Pytest fixes. * Pytest fixes, revert old test. * Move make_sure_path_exists before creating PipestatManager Object * Ensure pypiper is calling pipestat interface and NOT the backend. * Removed redundant default output schema function and refactored. * Add clear_status to end of unit test. * Implement pipestat.report for reporting results and objects with and without annotations. * Fix unit test, reporting results, and refresh_stats. #187 * lint * remove report_object and simplify report_result * Add report_object back for backwards compatibility. * Add ability to pass result_format to pipestat and return formatted string. Added annotation for backwards compatibility. * Change to passing formatting function to pipestat instead of format flag. * Added default mark down formatter to pass during pipestatmanager creation. Fixed sample_name and pipeline_name mix up for pipeline manager. * Add passing multi flag to pipestatmanager during creation. * Add printing pipestat arguments to log file. * Polish output of pipestat object after initialization * Update docs * Update requirements * fix _failed property return * update changelog
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This will be a major release that adds pipestat support.