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 overview.md #28

Merged
merged 4 commits into from
Jul 29, 2022
Merged

ADD overview.md #28

merged 4 commits into from
Jul 29, 2022

Conversation

andycon
Copy link
Contributor

@andycon andycon commented Jul 14, 2021

No description provided.

Copy link
Member

@yarikoptic yarikoptic left a comment

Choose a reason for hiding this comment

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

Initial pass with some suggestions

overview.md Outdated Show resolved Hide resolved
overview.md Outdated
A “record” in this context refers to a digital media file that contains the audio and visual stimulation presented to a subject while undergoing fMRI scanning for a particular session.
The goal is to provide one record for every functional acquisition collected at an fMRI research center.
The merits of collecting such records, if not obvious, are numerous and will be considered in a separate section.
While the scope of the project is limited to the special circumstances of collecting data at an fMRI research center, we anticipate that future modifications driven by interest and necessity may expand the set of use-cases to include other behavioral and neuroimaging paradigms.
Copy link
Member

Choose a reason for hiding this comment

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

with the above note, I think not even worth mentioning this here, and just say that every neuroimaging file (be it fmri, anat, dwi, etc) would have a corresponding stimulus file.

Also -- ultimate goal is to incorporate ReproStim within ReproIn/heudiconv data conversion, so to have all these files placed into BIDS dataset. Worth adding a reference to bids-standard/bids-specification#751 as WiP to possibly enhance BIDS to support per-neuroimaging file stimuli.

And also, such system could also be employed for purely behavioral experiments, and those are supported by BIDS as well.

overview.md Show resolved Hide resolved
overview.md Outdated Show resolved Hide resolved
Setting up ReproStim requires an initial investment in hardware, including a video capture device, a computer that runs the ReproStim capture server software,
a USB “sniffer” device to capture scanner trigger pulses for use by the capture service,
and necessary cables and connectors including audio and video splitter cables.
[These details will be filled in later, and there is a reasonable sketch of the hardware set up already in the README.md file.]
Copy link
Member

Choose a reason for hiding this comment

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

Ok for now, But as we build up this doc, most of the information would be moved from README.md into the docs, and README just to point to them.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK. So all details about st-up, configuration, etc. will be removed from the overview. Makes sense.

Copy link
Member

Choose a reason for hiding this comment

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

will it be done in this PR or you would like to do that in a separate follow up PR?

overview.md Outdated Show resolved Hide resolved
overview.md Show resolved Hide resolved
overview.md Show resolved Hide resolved
andycon and others added 2 commits August 26, 2021 12:17
Co-authored-by: Yaroslav Halchenko <[email protected]>
... Not just for functional acquisitions. added language to indicate intention to collect all stimulations for all types of scans, not just functional scans.
overview.md Outdated Show resolved Hide resolved
Co-authored-by: Yaroslav Halchenko <[email protected]>
@andycon
Copy link
Contributor Author

andycon commented Jul 29, 2022

Yarick says to merge this here now.

@andycon andycon merged commit 3517f8c into ReproNim:master Jul 29, 2022
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.

2 participants