-
Notifications
You must be signed in to change notification settings - Fork 0
fix: add examples to major, minor and patch changes #44
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
base: main
Are you sure you want to change the base?
Changes from 1 commit
ef41d98
025214b
81309e5
2514d40
b1be0db
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,8 +6,19 @@ Users need to understand how to interact with computed results produced by data | |
|
|
||
| Core data processing pipelines MUST adopt [semantic versioning](https://semver.org/). | ||
| - Major version changes indicate that the structure or interpretation of the data has changed. | ||
|
|
||
| - Update to `aind-data-schema` that renames or restructures the metadata. | ||
| - Any default parameter changes. | ||
| - Changes to the output file structure. | ||
|
|
||
| - Minor version changes indicate new, backwards compatible features were added to the pipeline. | ||
| - Patch version changes indicate bug fixes. | ||
|
|
||
| - Add a new parameter to the input arguments. | ||
| - Add a new QC plot. | ||
|
|
||
| - Patch version changes indicate backwards compatible bug fixes. | ||
|
|
||
| - Critical bug fixes that do not alter the data structure. | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'd say "do not alter output contents". Content vs structure is a big question here - I'd say writing to NWB 2.1 instead of 2.0 (changing structure in a non-breaking way) might make the most sense as a minor version (essentially bumping a dependency as in Sean's comment. but any change of content should be a major version (under strict semantic versioning anyway.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. also clearly an actual patch: bug fix that simply enables a job that was previously failing to generate results (since this isn't changing contents) |
||
|
|
||
| The pipeline's name and semantic version MUST be stored in aind-data-schema [Processing](https://github.com/AllenNeuralDynamics/aind-data-schema/blob/dev/src/aind_data_schema/core/processing.py#L970) metadata at the top level of the results. | ||
|
|
||
|
|
@@ -52,4 +63,4 @@ When querying the metadata database for `Processing.pipeline_version`, users and | |
| - Semantic versions `< 0.2.0` (i.e., `0.1.0`, `0.1.1`) | ||
| - Code Ocean versions from before semantic versioning was adopted (i.e., `18.0`) | ||
|
|
||
| For pipelines that have adopted semantic versioning, users and developers will always be able to find a pipelines semantic version in the `nextflow.config`. | ||
| For pipelines that have adopted semantic versioning, users and developers will always be able to find a pipelines semantic version in the `nextflow.config`. | ||
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.
add "any code update that changes output file content, even if that is fixing previously incorrect behavior"