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

CLI expects outdated processing status file column names #369

Closed
1 task done
alyssadai opened this issue Nov 1, 2024 · 1 comment · Fixed by #370
Closed
1 task done

CLI expects outdated processing status file column names #369

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

Comments

@alyssadai
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

Expected Behavior

No response

Current Behavior

Currently, bagel derivatives errors out when dealing with a Nipoppy-generated processing status TSV, about missing column names.

This is because the hardcoded column names the CLI is using is out of date with the current Nipoppy model for this file: https://nipoppy.readthedocs.io/en/latest/schemas/index.html#bagel-file

Error message

No response

Environment

  • OS:
  • Python/Node version:

How to reproduce

No response

Anything else?

No response
@alyssadai alyssadai added the type:bug Defects in shipped code and fixes for those defects label Nov 1, 2024
@alyssadai alyssadai moved this to Backlog in Neurobagel Nov 1, 2024
@alyssadai alyssadai self-assigned this Nov 1, 2024
@alyssadai alyssadai removed the type:bug Defects in shipped code and fixes for those defects label Nov 1, 2024
This was referenced Nov 1, 2024
Closed
Merged
@alyssadai alyssadai moved this from Implement - Active to Implement - Done in Neurobagel Nov 3, 2024
@github-project-automation github-project-automation bot moved this from Review - Active to Review - Done in Neurobagel Nov 4, 2024
@neurobagel-bot Neurobagel Bot
Copy link
Contributor

neurobagel-bot bot commented Nov 4, 2024

🚀 Issue was released in v0.3.1 🚀

@neurobagel-bot neurobagel-bot bot added the released This issue/pull request has been released. label Nov 4, 2024
@alyssadai alyssadai mentioned this issue Nov 4, 2024
Open
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
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[FIX] Add missing _id suffix to processing status file column mappings neurobagel/bagel-cli
1 participant
@alyssadai