-
Notifications
You must be signed in to change notification settings - Fork 56
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
Sign error in z coordinates of ContourData #24
Comments
Hi there, thank you for your kind words and for using our library! If you could, please feel free to submit a merge request for your changes and we can take a look and try to get it into the tool. We can probably incorporate your second request to the same pull request. |
Hi! Many thanks for the reply. I just found out that the error is due to the SliceLocation attribute that was discussed in other issues. The scans with the z sign problem all had reversed sign in SliceLocation compated to ImagePositionPatient[2]. I can make a pull request later where just the two lines with SliceLocation are updated to ImagePositionPatient[2]. |
Someone was working on an update to use ImagePositionPatient already https://github.com/qurit/rt-utils/pull/26/files. Do these changes solve your problem? I've also created a separate issue for the ROIGenerationAlgorithm if you would like to take a look #27 :) |
Yes changing to ImagePositionPatient will solve the problem! I will have a look at the ROIGenerationAlgorithm. |
Dear Authors,
thanks a lot for making this great library public, it is highly appreciated!
We have encountered a small problem with it: we're using 3d CT images and due to the left-handed coordinate system the z coordinates in the image are negative. However the ROIs created by your library are missing the minus sign. By manually hacking I managed to get it working now, but it would be nice if this could be solved in your code.
Also it would be nice if one could change the ROIGenerationAlgorithm to AUTOMATIC possibly just as an additional argument to add_roi.
Many thanks!
Thomas
The text was updated successfully, but these errors were encountered: