spinal-cord-7t / coil-qc-code

7T Spinal Cord Coil QC Analysis Code
https://spinal-cord-7t.github.io/coil-qc-code/README.html
1 stars 1 forks source link

Add coil element layout across coils #77

Open jcohenadad opened 5 months ago

jcohenadad commented 5 months ago

Given that this paper is about comparing coils, an obvious question when interpreting B1+ and B1- results is: how are the Tx and Rx coils distributed for each coil? We should gather this information from each site (whenever possible, given that some are proprietary), and create a figure with this. Example of layout figure:

image

We could use the same color coding to display coil arrangement. For example, Tx elements in red, Rx in blue and Tx/Rx in purple.

Ideally, we would display the STL file in the notebook for interactive 3D view of each coil layout.

List of sites to generate a model from:

πŸ‘‰ Centralized document with added information

honzink commented 4 months ago

@jcohenadad Just to double-check. This is the manuscript?

So I can mention title and co-authors, when I reach out to manufacturers. I will start when I come back from vacation end of July.

evaalonsoortiz commented 4 months ago

Yes, it is.

jcohenadad commented 4 months ago

So I can mention title and co-authors, when I reach out to manufacturers

it is better if I'm the point of contact with manufacturers-- these are very sensitive information that they shared and I prefer that they are not under the impression that I distributed it widely

honzink commented 3 months ago
Capture d’écran 2024-08-08 aΜ€ 11 49 32 Capture d’écran 2024-08-08 aΜ€ 14 35 19

Before I continue, this is for the coil in line # 2 and 4. Does that serve the purpose? They are CAD models so we can turn and shift and modify later.

It takes me some time to create them though, but it's the only way that I can think of.

jcohenadad commented 3 months ago

Before I continue, this is for the coil in line # 2 and 4. Does that serve the purpose?

Yes, that’s what I had in mind. We should sit down together so I can explain the fine details of how to display the elements.

One strange thing however, is that line #2, which is CRMBM, is an 8ch TxRx, which looks different than the what you showed

honzink commented 3 months ago

@jcohenadad I meant line # 2 in the centralized document. Or do we need only models for the ones you mentioned above?

jcohenadad commented 3 months ago

@jcohenadad I meant line # 2 in the centralized document. Or do we need only models for the ones you mentioned above?

Why are you modeling line #2 of the centralized document? It is not neededβ€” see the list:

MGH/MNI: https://pubmed.ncbi.nlm.nih.gov/37439129/ CRMBM: 8ch Tx/Rx (Rapid Biomedical) MPI: 24ch Tx/Rx (MRItools) MSSM: 4ch Tx 22ch Rx + 4ch Tx 18ch Rx (Custom) NTNU: Same as MPI UCL: Same as CRMBM

honzink commented 3 months ago

CRMBM: 8ch Tx/Rx (Rapid Biomedical)

Capture d’écran 2024-08-23 aΜ€ 11 35 23
honzink commented 3 months ago

MGH/MNI. I had to doctor the Rx array to fit the human body model. Coil 3D model from Nibardo.

Capture d’écran 2024-08-23 aΜ€ 16 06 01
honzink commented 3 months ago

Changes: -White background -continuous conductors -human body model with higher resolution. I haven't found a fast way to make the human model smoother, it might take a day maybe more. I personally think it's fine, but up to you.

Capture d’écran 2024-08-28 aΜ€ 17 23 03
jcohenadad commented 3 months ago

Much better! Yes, I agree the human model looks fine as is. I would just make it lighter so it contrasts better with the coil elements.

Also, would it be possible to have the coil elements appear as 3D wires, as opposed to as a flat surface? They would appear more clearly (eg: we don't see the upper most element very well depending on its orientation)

honzink commented 3 months ago

Grey gives a good contrast, and is skin colour neutral.

3D wires would take me a day. I thickened the surface which serves the purpose, I hope.

Capture d’écran 2024-08-29 aΜ€ 13 39 18
jcohenadad commented 3 months ago

Great! Would you be able to make the gray much lighter?

honzink commented 3 months ago

How is this?

Capture d’écran 2024-08-30 aΜ€ 11 18 26
honzink commented 3 months ago

This is MSSM:

Capture d’écran 2024-08-30 aΜ€ 11 49 22
jcohenadad commented 3 months ago

On the MSSM coil, are the two posterior purple elements Tx/Rx? If it is Tx-only, I would pick another color (eg red)

honzink commented 3 months ago

Yes, that's purple, in their MRM paper they are TR elements.

jcohenadad commented 3 months ago

maybe we should pick another blue or purple, to better distinguish the two (I find them very close in intensity). Maybe a slightly lighter blue?

honzink commented 3 months ago

How is this?

Capture d’écran 2024-08-30 aΜ€ 14 47 44
honzink commented 2 months ago

Here the paper figure. Let me know if you want any changes.

paper figure.pdf

a) MPI: 24ch Tx/Rx (MRItools), NTNU: Same as MPI b) CRMBM: 8ch Tx/Rx (Rapid Biomedical), UCL: Same as CRMBM c) MSSM: 4ch Tx 22ch Rx + 4ch Tx 18ch Rx (Custom) d) MGH/MNI: https://pubmed.ncbi.nlm.nih.gov/37439129/

red: Tx elements, blue: Rx only elements, purple: TR elements

jcohenadad commented 2 months ago

excellent, thank you @honzink.

honzink commented 2 months ago

How is this blue?

Capture d’écran 2024-09-23 aΜ€ 14 38 47
jcohenadad commented 2 months ago

How is this blue?

much better, thanks!

honzink commented 2 months ago

Here the four subfigures:

MRI_tools.pdf MSSM_.pdf neuropoly.pdf rapid.pdf

jcohenadad commented 1 month ago

@honzink for archiving purpose (eg: reproduce/modify figure in the future), I'd like to keep all figure sources inside a centralized location. Can you please point us to the location of the files with which you produced those figures? Thanks

evaalonsoortiz commented 1 month ago

Hi @honzink, one of the coil owners/developers has raised a concern about the display of the coil elements of their coil. See: https://docs.google.com/document/d/160YWLPuc6Rean6-BpXsHqRidOXrypX9Yt4fW89LzXPo/edit?tab=t.0#:~:text=superiorly.%C2%A0%20Could%20you-,check,-it%20again%20against

Could you follow up? We can discuss here and then move over to the google doc once there's a consensus.

honzink commented 1 month ago

I copied the figure sources to duke:

\duke.neuro.polymtl.ca\public\jan\paper_figure

evaalonsoortiz commented 1 month ago

There is another comment regarding a coil, I'm tagging you both on the google doc to simplify exchanges.

honzink commented 1 month ago

@evaalonsoortiz you tagged the wrong Jan, it should be jan.paska@polymtl.ca :)

I would suggest to send the respective coil figures to the commenters, so they can indicate in a sketch the changes they want to minimize a back and forth.

jcohenadad commented 1 month ago

@honzink I found this coil config below under Figures/Coils/CAD_files/MSSM/try3.SLDPRT, but it does not seem to correspond to the MSSM layout shown here https://github.com/spinal-cord-7t/coil-qc-code/issues/77#issuecomment-2321737127:

image

Can you please clarify what are all the SLDPRT files found here:

oils/CAD_files
β”œβ”€β”€ CRBMB
β”‚Β Β  β”œβ”€β”€ ODU cable holder.STL
β”‚Β Β  β”œβ”€β”€ coil12.SLDPRT
β”‚Β Β  β”œβ”€β”€ coil12_thick_conductor.SLDPRT
β”‚Β Β  β”œβ”€β”€ housing_scanner_side.stl
β”‚Β Β  β”œβ”€β”€ leiterbahn.stl
β”‚Β Β  └── ~$coil12_thick_conductor.SLDPRT
β”œβ”€β”€ MGH MNI
β”‚Β Β  β”œβ”€β”€ Coil_elements_20Rx-thick-wires (1).igs
β”‚Β Β  β”œβ”€β”€ Coil_elements_8Tx-20Rx.igs
β”‚Β Β  β”œβ”€β”€ coil mgh mni 1.SLDPRT
β”‚Β Β  β”œβ”€β”€ coil mgh mni.SLDPRT
β”‚Β Β  └── ~$coil mgh mni 1.SLDPRT
β”œβ”€β”€ MPI
β”‚Β Β  β”œβ”€β”€ C-Spine_2022_4_coilonly copie.stl
β”‚Β Β  β”œβ”€β”€ MRItools.SLDPRT
β”‚Β Β  └── ~$MRItools.SLDPRT
β”œβ”€β”€ MRI_tools.pdf
β”œβ”€β”€ MSSM
β”‚Β Β  β”œβ”€β”€ Magnetic Resonance in Med - 2016 - Zhang - 7 Tesla 22‐channel wrap‐around coil array for cervical spinal cord and brainstem.pdf
β”‚Β Β  β”œβ”€β”€ try1.SLDPRT
β”‚Β Β  β”œβ”€β”€ try2.SLDPRT
β”‚Β Β  β”œβ”€β”€ try3.SLDPRT
β”‚Β Β  β”œβ”€β”€ ~$try1.SLDPRT
β”‚Β Β  └── ~$try2.SLDPRT
β”œβ”€β”€ MSSM_.pdf
β”œβ”€β”€ coil3.SLDPRT
β”œβ”€β”€ human smooth.SLDPRT
β”œβ”€β”€ human smooth2.SLDPRT
β”œβ”€β”€ human smooth3.SLDPRT
β”œβ”€β”€ human.IGS
β”œβ”€β”€ human.SLDPRT
β”œβ”€β”€ male_highprecision.SLDPRT
β”œβ”€β”€ male_highprecision.iges
β”œβ”€β”€ male_highprecision_body.SLDPRT
β”œβ”€β”€ neuropoly.pdf
β”œβ”€β”€ old
β”‚Β Β  β”œβ”€β”€ coil2
β”‚Β Β  β”‚Β Β  β”œβ”€β”€ paper_coil2.IGS
β”‚Β Β  β”‚Β Β  └── paper_coil2.SLDPRT
β”‚Β Β  β”œβ”€β”€ coil4
β”‚Β Β  β”‚Β Β  └── coil4.SLDPRT
β”‚Β Β  β”œβ”€β”€ coil5
β”‚Β Β  β”‚Β Β  └── Magnetic Resonance in Med - 2019 - Rietsch - Development and evaluation of a 16‐channel receive‐only RF coil to improve 7T.pdf
β”‚Β Β  └── coil9
β”‚Β Β      β”œβ”€β”€ coil9.SLDPRT
β”‚Β Β      └── nihms-698231.pdf
└── rapid.pdf
jcohenadad commented 1 month ago

"try2" seems to be the correct file for MSSN, but let's wait for @honzink to clarify the meaning of all files, so we can document and do some cleanup