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

add --force mode for metadata.yml file #86

Merged
merged 16 commits into from
Jul 9, 2024
Merged

add --force mode for metadata.yml file #86

merged 16 commits into from
Jul 9, 2024

Conversation

julia-pfarr
Copy link
Member

Modification: so far it was ok not to pass a metadata.yml file. However, this file contains required metadata. Now the program stops if no metadata.yml file was passed and gives information on why the program stopped and how to fix that. Additionally, now it hase a --force flag where Users can still convert their edfs without passing a metadata.yml file. The user is then informed that the resulting dataset is invalid to BIDS.

Closes #83

@julia-pfarr julia-pfarr requested a review from Remi-Gau July 5, 2024 11:40
@julia-pfarr julia-pfarr added the enhancement New feature or request label Jul 5, 2024
eye2bids/edf2bids.py Outdated Show resolved Hide resolved
eye2bids/edf2bids.py Outdated Show resolved Hide resolved
Comment on lines 50 to 53
"""Load the metadata.yml file with the additional metadata.\n
This file must contain at least the additional REQUIRED metadata
You can find a template in the eye2bids GitHub.\n
This file must contain at least the additional REQUIRED metadata\n
in the format specified in the BIDS specification.\n"""
Copy link
Contributor

Choose a reason for hiding this comment

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

update message to let users know how to avoid this warning

eye2bids/edf2bids.py Outdated Show resolved Hide resolved
eye2bids/edf2bids.py Outdated Show resolved Hide resolved
@Remi-Gau Remi-Gau merged commit c587b13 into main Jul 9, 2024
6 checks passed
@Remi-Gau Remi-Gau deleted the metadata-yml branch July 9, 2024 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

metadata.yml
2 participants