Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Emit warning instead of erroring out when pipeline/version not recognized #382

Closed
alyssadai opened this issue Nov 7, 2024 · 1 comment · Fixed by #401
Closed

Emit warning instead of erroring out when pipeline/version not recognized #382

alyssadai opened this issue Nov 7, 2024 · 1 comment · Fixed by #401
Assignees
@alyssadai
Labels
released This issue/pull request has been released.

Comments

@alyssadai
Copy link
Contributor

alyssadai commented Nov 7, 2024 •
edited
Loading

As long as at least one pipeline and version combination is found in the pipeline catalog, the derivatives command should proceed without erroring out.
@alyssadai alyssadai added the flag:discuss Flag issue that needs to be discussed before it can be implemented. label Nov 7, 2024
@alyssadai alyssadai added the flag:schedule Flag issue that should go on the roadmap or backlog. label Nov 18, 2024
@surchs surchs removed the flag:schedule Flag issue that should go on the roadmap or backlog. label Nov 18, 2024
@surchs surchs moved this to Backlog in Neurobagel Nov 18, 2024
@alyssadai alyssadai changed the title Consider emitting warning instead of erroring out when pipeline/version not recognized Emit warning instead of erroring out when pipeline/version not recognized Nov 20, 2024
@alyssadai alyssadai moved this from Backlog to Specify - Done in Neurobagel Nov 28, 2024
@alyssadai alyssadai moved this from Specify - Done to Implement - Active in Neurobagel Nov 28, 2024
@alyssadai alyssadai self-assigned this Nov 28, 2024
@alyssadai alyssadai removed the flag:discuss Flag issue that needs to be discussed before it can be implemented. label Dec 3, 2024
@alyssadai alyssadai mentioned this issue Dec 3, 2024
Merged
7 tasks
@alyssadai alyssadai moved this from Implement - Active to Implement - Done in Neurobagel Dec 4, 2024
@surchs surchs moved this from Implement - Done to Review - Active in Neurobagel Dec 4, 2024
@github-project-automation github-project-automation bot moved this from Review - Active to Review - Done in Neurobagel Dec 11, 2024
@neurobagel-bot Neurobagel Bot
Copy link
Contributor

🚀 Issue was released in v0.3.5 🚀

@neurobagel-bot neurobagel-bot bot added the released This issue/pull request has been released. label Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alyssadai alyssadai

Labels
released This issue/pull request has been released.
Projects
Neurobagel
Status: Review - Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ENH] Allow unrecognized pipeline names and versions in processing status file neurobagel/bagel-cli
2 participants
@surchs @alyssadai