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

example microscopy data in the zarr format is missing a sidecar #385

Closed
robertoostenveld opened this issue Jun 25, 2023 · 6 comments · Fixed by #386
Closed

example microscopy data in the zarr format is missing a sidecar #385

robertoostenveld opened this issue Jun 25, 2023 · 6 comments · Fixed by #386

Comments

@robertoostenveld
Copy link
Collaborator

@effigies pointed me to this zarr example in this comment.

The specific subject and session comprises a SPIM (zarr) and a SEM (png) image. The zarr file has no sidecar JSON, although the spec states that to be required.

@effigies
Copy link
Contributor

Cc @TheChymera, who I think prepared this example.

@TheChymera
Copy link
Contributor

@robertoostenveld but it has a JSON file:

[deco]~/src/bids-examples/micr_SEMzarr/sub-01 ❱ ls */micr
ses-01/micr:
sub-01_ses-01_sample-A_SEM.json  sub-01_ses-01_sample-A_SEM.png  sub-01_ses-01_sample-A_SPIM.ome.zarr

ses-02/micr:
sub-01_ses-02_sample-A_SEM.json  sub-01_ses-02_sample-A_SEM.png
[deco]~/src/bids-examples/micr_SEMzarr/sub-01 ❱ cat ses-01/micr/sub-01_ses-01_sample-A_SEM.json
{
        "Manufacturer": "Hamamatsu",
        "ManufacturersModelName": "C4742-95-12G04",
        "BodyPart": "CSPINE",
        "SampleEnvironment": "ex vivo",
        "SampleFixation": "4% paraformaldehyde, 2% glutaraldehyde",
        "SliceThickness": 1,
        "TissueDeformationScaling": 98,
        "PixelSize": [0.18, 0.18],
        "PixelSizeUnits": "um"
}

@effigies
Copy link
Contributor

That's the png file that has a sidecar, not the microscopy file. The suffixes need to match.

@TheChymera
Copy link
Contributor

ah, ok.

@TheChymera
Copy link
Contributor

@effigies @robertoostenveld does this solve the issue?

#386

@robertoostenveld
Copy link
Collaborator Author

Yes. It is a very minimalistic sidecar, but as far as I can tell, it fulfills the requirements.

thanks!

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

Successfully merging a pull request may close this issue.

3 participants