Steve Pieper

Results 544 comments of Steve Pieper
trafficstars

Probably it's this part: https://github.com/SlicerMorph/SlicerMorph/blob/8af8a03a2ffdadec79cbc070e197e6f3daa26d86/ImageStacks/ImageStacks.py#L710-L712 I'm guessing this is a valid complaint for this data, but maybe not important in this use case. Maybe the best would be to fix...

Then probably the best solution is to fix ITK so it improves behavior on all systems then. That is, if you really think these are valid tiff that ITK should...

It would be great to be able to use dicom headers for lossless conversion between ome-ngff and WSI or other dicom objects. A lot of the concepts mentioned in this...

+1 to @lassoan's suggestion of adopting the `(scheme, value, meaning)` code tuple concept from DICOM. Another feature of it is that in addition to standardizing references to useful vocabularies like...

This is probably because fiber bundles are a subclass of model with a different storage node. I don't think this is necessarily a bug.

To clarify, this is on windows? Which version? And which version of Slicer(Morph)?

This looks like [a long-running issue in Qt](https://bugreports.qt.io/browse/QTBUG-62149?focusedCommentId=366202&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel). Looks like it should be fixed in our version but maybe not. The code we use relies on qt to do this...

It was being served from the gh-pages branch. ![image](https://github.com/SlicerDMRI/whitematteranalysis/assets/126077/2ba21913-cdbe-4257-aca0-6cec305926de) I've just switched ti to serve from `master` so there should be an update in a few minutes.

If you can migrate to readthedocs and serve the content from the master branch that would be great. Or migrate the style code into the master branch so that there's...

> @pieper @swederik are you guys able to help review/approve this PR? not sure about Markus's availability nowadays. I read through this but did not test it. If you are...