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

[ENH] Add use of session entity as example of encoding multi-site studies in a single dataset #1803

Merged
merged 3 commits into from
May 22, 2024
Merged

[ENH] Add use of session entity as example of encoding multi-site studies in a single dataset #1803

merged 3 commits into from
May 22, 2024

Conversation

yarikoptic
Copy link
Collaborator

@yarikoptic yarikoptic commented Apr 24, 2024 •
edited
Loading

  • add missing trailing )
  • point to specific issue within BIDS 2.0
  • mention that different session indicators could be used

relevant discussions: bids-standard/bids-2-devel#11

@yarikoptic yarikoptic requested review from effigies and tsalo April 25, 2024 20:11
@yarikoptic yarikoptic added the enhancement New feature or request label Apr 25, 2024
@yarikoptic yarikoptic changed the title Improve multi-site description of "workarounds" Improve multi-site description of "workarounds" to mention possible use of session entity Apr 25, 2024
tsalo
tsalo approved these changes Apr 26, 2024
View reviewed changes
Copy link
Member

@tsalo tsalo left a comment

Choose a reason for hiding this comment

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

This looks good to me.

@effigies effigies changed the title Improve multi-site description of "workarounds" to mention possible use of session entity [ENH] Add use of session entity as example of encoding multi-site studies in a single dataset May 22, 2024
@effigies effigies merged commit a0e8c31 into bids-standard:master May 22, 2024
23 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tsalo tsalo tsalo approved these changes

@effigies effigies effigies approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@yarikoptic @effigies @tsalo