fractal-analytics-platform / fractal-tasks-core

Main tasks for the Fractal analytics platform
https://fractal-analytics-platform.github.io/fractal-tasks-core/
BSD 3-Clause "New" or "Revised" License
11 stars 5 forks source link

Make acquisition metadata optional for Plate pydantic model #748

Open jluethi opened 3 weeks ago

jluethi commented 3 weeks ago

Our current Plate Pydantic model mistakenly makes acquisitions a required argument, while the spec actually describes it as an optional argument. We should make it Optional.

We apparently always add the acquisition info to our plates. But e.g. the FAIM-IPA converters don't and it's spec compliant not to add it.

It is used only once in our code so far here: https://github.com/fractal-analytics-platform/fractal-tasks-core/blob/4165dc7b16dc159b12e0b6378c342681c790c95a/fractal_tasks_core/tasks/copy_ome_zarr_hcs_plate.py#L91