Open naga-karthik opened 4 months ago
If it is under labels_softseg_bin, the name softseg is not the right one, you can check in spine generic data multi subject in my branch the name convention we decided
I would go with C-A version since the 2.0 is not in sct, is that right?
Right, I checked and updated folder structure in my comment! The filenames are like this now:
sub-CXXX_UNIT1_desc-softseg_label-SC_seg.nii.gz
I would go with C-A version since the 2.0 is not in sct, is that right?
Correct, it's not in SCT. The 2.0
is essentially coming from our latest release. (but, technically, even this is not the soft_bin
model, but I think it's okay, the difference with the original soft
model isn't much.
Maybe we should create a release (like v.1?)
For nih-ms-mp2rage
I have generated these JSON files
{
"GeneratedBy": [
{
"Name": "contrast-agnostic-softseg-spinalcord",
"Version": "2.0"
},
{
"Author": "Nilser Laines Medina",
"Date": "2023-12-14",
"Note": "Binarised spinal cord soft segmentation"
}
]
}
Thanks @Nilser3 for your JSON example! I think I will go with contrast-agnostic-softseg-spinalcord
as the Name
but with a different version.
@sandrinebedard Sure, we can create a new release v2.1
today and I will be using this for the JSON sidecars
For
nih-ms-mp2rage
I have generated these JSON files{ "GeneratedBy": [ { "Name": "contrast-agnostic-softseg-spinalcord", "Version": "2.0" }, { "Author": "Nilser Laines Medina", "Date": "2023-12-14", "Note": "Binarised spinal cord soft segmentation" } ] }
The field "Name" is missing in this example. From where the second step was generated ? @Nilser3
Thanks for the feedback @NathanMolinier I see that it was still not in agreement with the new convention, I think it would be better something like:
{
"SpatialReference": "orig",
"GeneratedBy": [
{
"Name": "contrast-agnostic-softseg-spinalcord",
"Version": "2.0"
},
{
"Name": "Manual",
"Author": "Nilser Laines Medina",
"Date": "2023-12-14",
"Note": "Binarised spinal cord soft segmentation"
}
]
}
Just a nitpick, we concluded here that we should use yyyy-mm-dd hh:mm:ss
format for Date
to make it easy to distinguish the order of corrections.
{ "SpatialReference": "orig", "GeneratedBy": [ { "Name": "contrast-agnostic-softseg-spinalcord", "Version": "2.0" }, { "Name": "Manual", "Author": "Nilser Laines Medina", "Date": "2023-12-14", "Note": "Binarised spinal cord soft segmentation" } ] }
Just out of curiosity, did you really binarize manually ? Or did you use a custom script doing thresholding ?
was binarized sct_maths
after generating the soft masks,
but, I think I will remove this "Note" , because I will generate again these SC masks with the last version of contrast-agnostic-model
(there the result is already binary).
was binarized
sct_maths
after generating the soft masks,
You should then specify the method sct_maths
using this "Name"
field instead of Manual, and potentially provide the command you ran, like below:
{
"SpatialReference": "orig",
"GeneratedBy": [
{
"Name": "contrast-agnostic-softseg-spinalcord",
"Version": "2.0"
},
{
"Name": "sct_maths",
"Param": "-thr 0.8",
"Version": "SCT v6.2",
"Note": "Binarised spinal cord soft segmentation"
}
]
}
It's pretty cool that you can directly define the json dict and sct_run_batch
script and can create the json file for each subject with the contents of the json dict.
I am working on adding new datasets/contrasts to augment the existing spine-generic dataset. I wanted to clarify/confirm a few things about the naming conventions to be used for the outputs derived from the contrast agnostic model (which will be used for training)
Example folder structure for `basel-mp2rage`
``` basel-mp2rage ├── README.md ├── dataset_description.json ├── participants.tsv ├── participants.json ├── code/ ├── derivatives │ └── labels │ ├── dataset_description.json │ ├── README.md │ └── sub-CXXX │ └── anat │ ├── sub-CXXX_UNIT1_label-SC_seg.nii.gz │ ├── sub-CXXX_UNIT1_label-SC_seg.json │ └── labels_softseg_bin ---> this folder is newly added │ ├── dataset_description.json │ ├── README.md │ └── sub-CXXX │ └── anat │ ├── sub-CXXX_UNIT1_desc-softseg_label-SC_seg.nii.gz │ ├── sub-CXXX_UNIT1_desc-softseg_label-SC_seg.json ├── sub-CXXX │ └── anat │ ├──sub-CXXX_UNIT1.nii.gz │ └──sub-CXXX_UNIT1.json │ ```Contents of the json file
Issue is that the model currently in
SCT v6.2
is the originalsoft
model trained on soft GTs. But, the model I will be use for inference and training with new contrasts is thesoft_bin
model.EDIT: updated the filename for
labels_softseg_bin