strict_well 0.5: allOf should have two or more children

[toloudis]

Currently the 0.5 version of strict_well has an allOf with only a single ref.

ngff/schemas/strict_well.schema

Line 3 in 8cbba21

"allOf": [

This is giving errors when trying to convert with zod as in zod-ome-ngff
It looks like the following might be missing (copying from the 0.4 spec) -- this makes zod-ome-ngff work but I have not yet confirmed that it is 100% correct.

,
    {
      "properties": {
        "well": {
          "required": [
            "version"
          ]
        }
      }
    }

[will-moore]

As far as I can see, the only difference between strict_well and regular well for v0.4 was the requirement for version, but now that there is no version in v0.5 (because it's moved to the top-level attributes/ome) there is no difference between strict_well and well in v0.5?

[d-v-b]

Currently the 0.5 version of strict_well has an allOf with only a single ref.

ngff/schemas/strict_well.schema

Line 3 in 8cbba21
"allOf": [

This is giving errors when trying to convert with zod as in zod-ome-ngff
It looks like the following might be missing (copying from the 0.4 spec) -- this makes zod-ome-ngff work but I have not yet confirmed that it is 100% correct.

,
    {
      "properties": {
        "well": {
          "required": [
            "version"
          ]
        }
      }
    }

are you generating zod schemas from the ome-zarr JSON schema documents directly?

[toloudis]

are you generating zod schemas from the ome-zarr JSON schema documents directly?

I was trying to see how easy it would be to update https://github.com/manzt/zod-ome-ngff to support 0.5, and the first error I hit was this one.

[d-v-b]

@toloudis gotcha, just be advised that the JSON schema doesn't always match the text of the spec, and even if the JSON schemas were 100% accurate, there are invariants defined in the spec that JSON schema cannot express.