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

[XA61] physio in DICOM : sequence does not start #367

Open
benoitberanger opened this issue Jun 6, 2024 · 19 comments
Open

[XA61] physio in DICOM : sequence does not start #367

benoitberanger opened this issue Jun 6, 2024 · 19 comments
Assignees

Comments

@benoitberanger
Copy link

Hi @eauerbach,

Magnet : 3T Cima.X in XA61
Protocol : default + GLM disabled

When the Physio recording is in "DICOM" mode, the sequence does not start.
The message : "Scanning stopped by image reconstruction"
I did a SaveLog right after the crash. And the Utrace.
The physio devices were not in the magnet room, still on the wall charger.

Best,
Benoît

@julfou81
Copy link

julfou81 commented Jul 2, 2024

Hi @benoitberanger ,

From the tests from yesterday on the CIMA.X, I remember that the CMRR ep_2d_bold crashes on the CIMA.X in XA61 when whichever format was requested for Physio Logging: File, DICOM or Both, even though the respiratory signal measured by the spine coil was present. Do you confirm?

@benoitberanger
Copy link
Author

Just checked :

  • Off -> works
  • DICOM -> crashed at the beginning
  • File -> crashed at the beginning
  • Multiple -> crashed at the beginning
  • Legacy -> works
    I used the 64 channel coils, on a phantom, so no physio signals at all to record.

On a Prisma in VE11E, even though there is no signal to record, it works, the sequence starts and behaves normally, just the DICOM file is empty.

@julfou81
Copy link

julfou81 commented Jul 24, 2024

Same issue as with @benoitberanger with a Prisma after we upgraded to XA60 with release CMRR_MB_EPI_XA60A_R017pre9

@DanielGuellmar
Copy link

DanielGuellmar commented Jul 29, 2024

I can confirm the issue with a Prisma upgraded from XA60A to XA30A with the release CMRR_MB_EPI_XA60A_R017pre9 . Only the legacy mode writes data out to the Physio folder (so it behaves as expected by the setting File).

@eauerbach
Copy link
Member

I guess Siemens changed the format of the physiology data again, which would cause my interpreting code to throw an error. If someone can send me a savelog I will have a look at it.

@eauerbach eauerbach self-assigned this Jul 29, 2024
@julfou81
Copy link

julfou81 commented Jul 30, 2024

Thank you @eauerbach for looking into this issue!

Here is the UTraceSrv.utr file from our system (3T Prisma in XA60) after I launched several MB GRE-EPI acquisitions:

  • measID 25: bold sequence with Physio recording: DICOM and no physio input (crashed at launching)
  • measID 27: bold sequence with Physio recording: DICOM and physio input from the PMU simulation (crashed at launching)
  • measID 29: bold sequence with Physio recording: File and and physio input from the PMU simulation (crashed at launching)
  • measID 31: bold sequence with Physio recording: Legacy and and physio input from the PMU simulation (stopped by user because I put too many volumes)
  • measID 33: bold sequence with Physio recording: Legacy and and physio input from the PMU simulation (succeed after the acquisition of 3 volumes)

https://sdrive.cnrs.fr/s/a43W6AR7XX5mG3y

@eauerbach
Copy link
Member

Ok, it looks like there are two different issues: one seems to only affect XA61, and the other probably affects XA50/60/61. I have built some updated ICE binaries, can you try this and let me know how it goes? Just copy the two files to %CustomerIceProgs%, overwriting the existing files, then reboot the system. If there is still an error with this version, please send me an updated savelog (this version does more detailed logging).

XA60
XA61

@julfou81
Copy link

The problem is solved for XA60 with the new ICE binaries! Thank you @eauerbach !

@benoitberanger
Copy link
Author

Cima.X @ XA61
New binaries installed.

It works for the default protocol (+ GLM::Off). All Physio options works.

On an already running protocol (no the default, here with MB, ME, GRAPPA, etc) , Preparation of Image Reconstruction System has failed.

@benoitberanger
Copy link
Author

OK ok I think found the problem : SENSE1 option makes the sequence to NOT start.
Default prot (+glm::off) + SENSE1::Off -> OK
Default prot (+glm::off) + SENSE1::On -> Preparation of Image Reconstruction System has failed

MB option, ME option, Physio, everything seems to work (I didn't try with a real subjecto for the physio, but at least the option works).
Except the SENES1 option.

@benoitberanger
Copy link
Author

The first Error in the UTrace :

Exception during call of 'CMRR_MBIcePrg2D_NS_ejaIcePrg2DConfigurator::Compose()'
inner exception node : ejaCalcSenssProfile: SetProperty - Parameter not found UseExternalPhases

@eauerbach
Copy link
Member

This ICE patch can only be used with the R017pre9 version, did you already have that installed? I think that SetProperty error would appear if you installed the patch atop an older version.

@benoitberanger
Copy link
Author

To check for this specific info, I need to check with my colleagues who installed the sequence.
At the moment, I can have this info from dcm2niix conversion :

"PulseSequenceDetails": "%CustomerSeq%\\cmrr_mbep2d_bold",
"WipMemBlock": "b1993fb5-6970-4982-a023-f767275fba1d||Sequence: R017 nxva61a/main r/ed3a492e1; Apr 26 2024 18:01:22 by eja",

@eauerbach
Copy link
Member

That is indeed an older version; R017pre9 for XA61 should be dated May 21 2024. So you would need to install R017pre9 first, then follow with the patch.

@benoitberanger
Copy link
Author

Ok thanks, we will try and let you know.

@benoitberanger
Copy link
Author

Sorry for the very late response.
I just updated with the new binaries from May 21 : exactly the same results.

Off -> works
DICOM -> crashed at the beginning
File -> crashed at the beginning
Multiple -> crashed at the beginning
Legacy -> works
I used the 64 channel coils, on a phantom, so no physio signals at all to record.

Default protocol with GLM & MotionCorrection Off.

I'll upload the SaveLog soon.

@awalls77
Copy link

awalls77 commented Sep 26, 2024 via email

@eauerbach
Copy link
Member

Sorry for the very late response. I just updated with the new binaries from May 21 : exactly the same results.

Thanks, I had a look at the savelog and indeed I can see that you updated to the binaries from May 21, but after that you would also need to install the patch from here. Then you would have the sequence from May 21 and the ICE from July 30 and hopefully that combination works. It seems to work for XA60, so if I can get confirmation it also works for XA61 I will roll it all into a new proper release version.

@benoitberanger
Copy link
Author

As you expected it works now ! With the Sequence from May 21 and Ice from July 30 it works.
I checked with the default protocol and with a normal protocol : all physio option generate the _PhysioLog series.
I didn't check the content, but I expect it to be ok, since the problem was the sequence started and stop in less than a second.

If you need me to make more tests, please le me know. I'll send you the SaveLog and UTrace.

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

5 participants