-
Notifications
You must be signed in to change notification settings - Fork 157
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
[ENH] Clarify run indexing information for MRI acquisitions #719
Conversation
Are the CI failures expected? Do they need to be addressed separately? |
No that's weird. I'm guessing it's the result of #610 interacting with #714. I will try to fix. EDIT: I've figured it out. In #610 we switched to using the entity names, rather than keys, in the datatype YAML files, while in #714 we changed those names for a couple of entities. I'll open a PR to fix. |
…ata.md Co-authored-by: Stefan Appelhoff <stefan.appelhoff@mailbox.org>
a197fd9
to
b9d9b7b
Compare
@dlevitas Does this work for you? |
This looks good to me! |
Great! If you haven't already, could you go ahead and add yourself to the recent contributors? This will make sure we credit you on the next release for your contribution. |
Thanks everybody! |
Replaces #694.