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 master #6

Merged
merged 4 commits into from
Nov 23, 2020
Merged

Conversation

agahkarakuzu
Copy link
Collaborator

Pull the latest upstream.

effigies and others added 4 commits November 17, 2020 09:55
* MNT: Relax line length limit in YAML

* MNT: Ignore pycache in tools dir

* CI: Update configuration based on Travis validator

Also bump Python version and cache pip
* Add derivatives entities.

* Apply suggestions from code review

Co-authored-by: Chris Markiewicz <effigies@gmail.com>

* Differentiate derivative entities from raw entities.

* Update entities page.

Co-authored-by: Chris Markiewicz <effigies@gmail.com>
* Add part entity.

* Add self to contributors file.

* Address @yarikoptic's review.

* Fix entity table spacing.

* Add missing OPTIONAL for part under func (bold phase).

* Fix spacing again.

* Remove OPTIONAL under func (bold phase). Wasn't missing.

* Fix entity order (part goes after run).

* Fix typo.

* Incorporate part into schema.

* Update definition and examples to match BEP001.

* Fix code snippets.

* Fix filename patterns now that real/imag is supported.

* Apply suggestions from code review

- Change radians to rad.
- Treat `part` like `run`. Can only be omitted if only magnitude image is provided.

Co-authored-by: Chris Markiewicz <effigies@gmail.com>

* Support part with BOLD and update entity table.

* Update src/04-modality-specific-files/01-magnetic-resonance-imaging-data.md

Co-authored-by: Chris Markiewicz <effigies@gmail.com>

* Update entity table and entity definitions.

* Add link in text.

* Remove section header.

* Address review.

* Update src/04-modality-specific-files/01-magnetic-resonance-imaging-data.md

* Deprecate phase suffix.

* Fix link.

* Fix link again.

* Include deprecation in table.

* Fix table.

* Update 01-magnetic-resonance-imaging-data.md

* Apply suggestions from code review

Co-authored-by: Chris Markiewicz <effigies@gmail.com>

* Add entity field.

* Fix bug and generate appendix.

* Fix table.

* Update src/04-modality-specific-files/01-magnetic-resonance-imaging-data.md

Co-authored-by: Chris Markiewicz <effigies@gmail.com>

* Improve part definition.

Co-authored-by: Chris Markiewicz <effigies@gmail.com>
Co-authored-by: Chris Markiewicz <markiewicz@stanford.edu>
* [ENH] Add flip entity

- Modify `echo` entity respectively

* revert entities.md and update the schema

* Use schema tools

* [FIX] Format field

* Update src/schema/entities.yaml

Co-authored-by: Chris Markiewicz <effigies@gmail.com>

* Update src/schema/entities.yaml

Co-authored-by: Chris Markiewicz <effigies@gmail.com>

* Wording suggestion by @tsalo

* [ENH] Add flip entity

- Modify `echo` entity respectively

* revert entities.md and update the schema

* Use schema tools

* [FIX] Format field

* Update src/schema/entities.yaml

Co-authored-by: Chris Markiewicz <effigies@gmail.com>

* Update src/schema/entities.yaml

Co-authored-by: Chris Markiewicz <effigies@gmail.com>

* Wording suggestion by @tsalo

Co-authored-by: Chris Markiewicz <effigies@gmail.com>
@agahkarakuzu agahkarakuzu merged commit 436791d into bids-bep001:entity-inv-mt Nov 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants