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

GroupwiseRegistration notebook: ERROR: the direction cosines matrix of the fixed image is invalid! #291

Open
thewtex opened this issue May 22, 2024 · 5 comments

Comments

@thewtex
Copy link
Member

thewtex commented May 22, 2024

Error:

itk::ExceptionObject (0x562ce335dd80)
130
Location: "unknown" 
131
File: /work/build/cp38-cp38-manylinux_2_28_x86_64/_deps/elx-src/Components/Metrics/VarianceOverLastDimension/elxVarianceOverLastDimensionMetric.hxx
132
Line: 71
133
Description: ITK ERROR: VarianceOverLastDimensionMetric(0x562cdaf45be0): 
134
ERROR: the direction cosines matrix of the fixed image is invalid!
135

136
  The VarianceOverLastDimensionMetric expects the last dimension to represent
137
  time and therefore requires a direction cosines matrix of the form:
138
       [ . . 0 ]
139
  dc = [ . . 0 ]
140
       [ 0 0 1 ]

@N-Dekker any ideas?

@N-Dekker
Copy link
Collaborator

@thewtex Thanks for asking! This check was added by commit SuperElastix/elastix@e9a02d9 (Aug 6, 2014). Can you possibly still print the direction matrix of the fixed image, to get more details? I mean: this->GetElastix()->GetFixedImage()->GetDirection() at https://github.com/SuperElastix/elastix/blob/ef03c8d1795ed78e889e273fc8d3b93c6da53683/Components/Metrics/VarianceOverLastDimension/elxVarianceOverLastDimensionMetric.hxx#L59

@thewtex
Copy link
Member Author

thewtex commented Jun 20, 2024

@N-Dekker thanks for taking a look! 🙏

The direction is:

1 0 0
0 1 0
0 0 -1

(note the -1).

@N-Dekker
Copy link
Collaborator

N-Dekker commented Jul 8, 2024

@stefanklein @mstaring Do you possibly have a clue?

@mstaring
Copy link
Collaborator

mstaring commented Jul 8, 2024

the -1 should be a 1, as this is groupwise registration

@thewtex
Copy link
Member Author

thewtex commented Jul 9, 2024

I am not sure why the associated notebook test started to fail:

https://github.com/InsightSoftwareConsortium/ITKElastix/actions/workflows/build-test-package.yml

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

No branches or pull requests

3 participants