Skip to content

Conversation

@Rdornier
Copy link
Contributor

@Tom-TBT
Copy link
Contributor

Tom-TBT commented Nov 13, 2025

Ola,
I think you can leave RUN in the exclude key, that's an alias used around for PLATEACQUISITION.
There's always the case someone uses RUN in their csv.

And I think you mixed up with changes on the Split_View script for which you made another PR.

Cheers,
Tom

@Rdornier
Copy link
Contributor Author

Hello Tom,

Thanks for reviewing !

And I think you mixed up with changes on the Split_View script for which you made another PR.

Arrh, you're right. I've fixed it, thanks

@sbesson sbesson requested a review from Tom-TBT November 14, 2025 08:37
Copy link
Contributor

@Tom-TBT Tom-TBT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works as expected, lgtm

@sbesson
Copy link
Member

sbesson commented Nov 18, 2025

I assume https://omero-guides.readthedocs.io/en/latest/scripts/docs/parameter_description.html#include-parent-container-names should be updated accordingly if we get this included in the next OMERO.server release ? /cc @jburel @pwalczysko

@Rdornier
Copy link
Contributor Author

Hi @sbesson, Yes, you're right. This is now done. Thanks!

@sbesson sbesson closed this Nov 25, 2025
@sbesson sbesson reopened this Nov 25, 2025
@sbesson sbesson merged commit 87f4052 into ome:develop Nov 25, 2025
1 check passed
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 this pull request may close these issues.

Import_from_csv script : PlateAcquisition parent added as KVPs

3 participants