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

Error Computing Angle between 2 lines that include a midpoint of that missing landmark when the T1/T2 box is checked #82

Open
HUTIN1 opened this issue Aug 5, 2022 · 1 comment
Labels
AQ3DC Related to the AQ3DC module

Comments

@HUTIN1
Copy link
Contributor

HUTIN1 commented Aug 5, 2022

Somewhere in the code, that I could not identify where, there is an argument that skips a measurement for a landmark if this landmark is missing for a case/patient in a folder of patients' landmark lists. The issue is for computing Angle between 2 lines that include a midpoint of that missing landmark when the T1/T2 box is checked : the computation of the measurements with the midpoint fail for all cases if a single case does not have the landmark. We get the following keyerror : please see screenshot.

The users are going around this bug by computing Angle between 2 lines that include a midpoint of that missing landmark only at T, and then computing it again at T2, and in this case of computing it for a single time point , the code knows to skip the case that does not have the landmark.

Screenshot from 2022-08-05 14-48-51

@HUTIN1 HUTIN1 changed the title Error Angle between 2 lines that include a midpoint of that missing landmark when the T1/T2 box is checked Error Computing Angle between 2 lines that include a midpoint of that missing landmark when the T1/T2 box is checked Aug 5, 2022
@luciacev
Copy link
Contributor

luciacev commented Aug 9, 2022

@HUTIN1 @allemangD @jcfr @lucanchling @bianchijonas1
I just realized we forgot to note that this issue is regarding the AQ3DC code and not Q3DC. :-) Thanks!

@allemangD allemangD added the AQ3DC Related to the AQ3DC module label Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AQ3DC Related to the AQ3DC module
Projects
None yet
Development

No branches or pull requests

3 participants