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

Brainbrowser 2.5.3: (partial) Bug fix for HDF5 reader: Redmine 13063 #3068

Merged
merged 1 commit into from
Aug 23, 2017

Conversation

MounaSafiHarab
Copy link
Contributor

Upgrade to Brainbrowser 2.5.3 from 2.5.2 since CBRAIN team just released a version that fixes a bug in the HDF5 reader.

@MounaSafiHarab MounaSafiHarab added the Category: Bug PR or issue that aims to report or fix a bug label Aug 22, 2017
@MounaSafiHarab MounaSafiHarab added this to the 17.1 milestone Aug 22, 2017
@MounaSafiHarab
Copy link
Contributor Author

@gluneau
I assigned this to you as you have been the one upgrading brainbrowser all along.
bb2.5.3 just got released (minor bug fix) so I updated the 2 relevant files.

@MounaSafiHarab
Copy link
Contributor Author

MounaSafiHarab commented Aug 22, 2017

The issue is resolved for most scan types, but NOT for DTI. This issue is raised again here and is being investigated:
aces/brainbrowser#332

@MounaSafiHarab MounaSafiHarab changed the title Brainbrowser 2.5.3: Bug fix for HDF5 reader: Redmine 13063 Brainbrowser 2.5.3: (partial) Bug fix for HDF5 reader: Redmine 13063 Aug 22, 2017
@gluneau gluneau self-requested a review August 23, 2017 14:35
Copy link
Contributor

@gluneau gluneau left a comment

Choose a reason for hiding this comment

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

Testing the BB reports version:
BrainBrowser Volume Viewer v2.5.3

Tested on T1, T2, Bold, DTI, DTI65 of IBIS data set.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Bug PR or issue that aims to report or fix a bug Passed manual tests PR has been successfully tested by at least one peer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants