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

Save source and detector locations for 3D data #6

Merged
merged 1 commit into from
Apr 27, 2021

Conversation

rob-luke
Copy link
Member

@rob-luke rob-luke commented Jul 7, 2020

This PR addresses the issue described in #5. Please let me know if the current behaviour is expected and I will close that issue and this PR.

This PR stores source and detectors in obj.detectorPos3D instead of obj.detectorPos2D if 3 dimensions are available. It also writes the 3d variables to disk if they exist.

@rob-luke
Copy link
Member Author

@dboas @fangq @jayd1860 is there any way I can assist in merging the associated PR or closing this issue if its expected behaviour?

@dboas
Copy link

dboas commented Apr 27, 2021

@jayd1860 can you take care of this pull request?

@sstucker sstucker merged commit 6c1619a into fNIRS:master Apr 27, 2021
@sstucker
Copy link
Collaborator

Soon I will update this repo with code from Homer3 itself, some of which may affect the contents of the PR. Thank you for this contribution!

@rob-luke
Copy link
Member Author

Thank you for considering this PR

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