Skip to content

correction request for the LocaliZoom content type #278

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

Open
Majpuc opened this issue Mar 25, 2025 · 5 comments · May be fixed by #288
Open

correction request for the LocaliZoom content type #278

Majpuc opened this issue Mar 25, 2025 · 5 comments · May be fixed by #288
Assignees
Labels
ContentType ContentType instances - addition of new or update of existing ones

Comments

@Majpuc
Copy link

Majpuc commented Mar 25, 2025

Dear all,

I noticed that the cc for LocaliZoom was missing the last part. I think it should be "application/vnd.localizoom.lz+json"

Could you make this correction?

Thank you!

Best, Maja

@UlrikeS91 UlrikeS91 self-assigned this Mar 25, 2025
@UlrikeS91 UlrikeS91 added the ContentType ContentType instances - addition of new or update of existing ones label Mar 25, 2025
@UlrikeS91
Copy link
Contributor

Thanks for raising the issue. I see that the LocaliZoom content type is also missing other properties. Would it make sense to do a proper update of the content type or do you need the corrected content type available quickly?

@Majpuc
Copy link
Author

Majpuc commented Mar 25, 2025

Yes, I agree. We could add "description": "JSON specific file for LocaliZoom containing information about the reference atlas "target" and "target-resolution" and section image registration information for each section in the series."
I am not sure about the "dataType" and "displayLabel". Maybe Gergely can supply info here.

@UlrikeS91
Copy link
Contributor

Just to clarify: Does LocaliZoom create one output format with file extension ".lz" (which may or may not be based on JSON format) or is there an additional one with ".json"?

The logic "application/vnd.localizoom.lz+json" as name combined with ".lz" as file extension does not work for our instances.

It can either be:
"application/vnd.localizoom.lz" (name) + ".lz" (file extension)
or
"application/vnd.localizoom+json" (name) + ".json" (file extension)

If both exist, both will be added of course.

Comparable to e.g.,
application_vnd.quicknii.flat with ".flat" as file extension
vs.
application_vnd.quicknii+json with ".json" as file extension

@Tevemadar
Copy link

Historically both exists. As of now, LocaliZoom in the Collab creates .lz files, but from the past we have the same kind of files with .json extension, they may appear in datasets.

@Majpuc
Copy link
Author

Majpuc commented Apr 15, 2025

Hi @UlrikeS91 Could you create both please ?

@UlrikeS91 UlrikeS91 linked a pull request May 2, 2025 that will close this issue
@UlrikeS91 UlrikeS91 linked a pull request May 2, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ContentType ContentType instances - addition of new or update of existing ones
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants