-
Notifications
You must be signed in to change notification settings - Fork 14
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
Add test case: Save image according to ome-ngff standards #73
Comments
Hi @ClementCaporal , awesome idea! We already have a test case for loading zarr files: https://github.com/haesleinhuepf/human-eval-bia/blob/7e9712670168b71bea44e17e7389dfc120dcc96e/test_cases/open_zarr.ipynb Do you think you could formulate a similar task for saving ngff files? Thanks for the suggestion! |
Hello, Sorry for the delay, I am discovering how frustrating it can be to create a good prompt for a "simple" task. Meanwhile I did this notebook that just check if it is able to save a zarr file: https://gist.github.com/ClementCaporal/02a9401877c2b8b49fe6edbca5816962 |
Hi @ClementCaporal , awesome, the test-case looks good to me! Consider sending a pull-request. You can also later add a more complicated test-case for zarr files. And please do not make any decisions on how to change the test case depending on how individual LLMs solve the test. Ideallly you write a test-case without promoting any LLM. Otherwise you would bias the test 😉 |
Note for later, this repo could be a good solution to check if the created zarr respect the ngff specification: https://github.com/BioImageTools/ome-zarr-models-py |
I don't know if this is a well-defined "biological image analysis code" task, but ensuring that images produced by LLM are saved in ome/ome-ngff format to be compatible with other scripts may be something important?
As the LLM might not have a lot of training with this format, describing it at the beginning inside the prompt might be necessary? So it can at least put the right dimension order
The text was updated successfully, but these errors were encountered: