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.

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

Update to match neurodatascience/nipoppy:main #77

Merged
merged 1 commit into from
Nov 17, 2023
Merged

Conversation

michellewang
Copy link
Collaborator

PR created/updated automatically by create-pull-request GitHub action. Will be merged automatically by automerge-action if there are no merge conflicts.

* use manifest instead of doughnut to get participants to track

* fix identical bagel still being written

* Check `tar` and `tar.gz` extensions

* add HAS_DATATYPE__{} columns to bagel

* fix warning in pd.DataFrame.compare()

* change logic for UNAVAILABLE status based on manifest

* commit dashboard schema

* fix warning about df_bagel_old_full not being defined

* refactor based on Nikhil comments in Slack meeting
Copy link
Collaborator

@bcmcpher bcmcpher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR approved by auto-approve-action GitHub action.

@michellewang michellewang merged commit 9ec79bf into main Nov 17, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants