Skip to content
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

Try omero_ext.path to import the path module #7

Merged
merged 2 commits into from
Nov 28, 2019

Conversation

sbesson
Copy link
Member

@sbesson sbesson commented Nov 26, 2019

@dominikl
Copy link
Member

dominikl commented Nov 28, 2019

I tried to run the plugin (pip installed from snoopy repo). Seems to work:

./omero -s merge-ci-devspace.openmicroscopy.org -u user-3 upload OMERO-upload-test.txt 
Using session for user-3@merge-ci-devspace.openmicroscopy.org:4064. Idle timeout: 10 min. Current group: read-only-1
OriginalFile:386794

But: Then I tried to find the file in Insight. Not there. Can't see it in attachments, or coming up in the list of 'uploaded files' when I try to create a file attachment. Is there a way apart from sql/hql to find the file?

@sbesson
Copy link
Member Author

sbesson commented Nov 28, 2019

I am unsure an OriginalFile might be accessible as such via the UI, we might want some feature allowing to wrap it as a FileAnnotation

@joshmoore
Copy link
Member

👍 That would have simplified https://forum.image.sc/t/end-of-life-dih-software-from-slidepath-leica-recovery-to-omero/31276/6?u=joshmoore quite a bit. (Additionally attaching on import would be another RFE)

@dominikl
Copy link
Member

Ah thanks. That's true, it needs to be a FileAnnotation in order to show up in Insight.
There it is now:
Screen Shot 2019-11-28 at 11 41 06

@sbesson
Copy link
Member Author

sbesson commented Nov 28, 2019

Note that none of the discussion above is caused by this PR :) Should we get this merged and look into expanding the library/CLI plugin as follow-up?

@sbesson sbesson merged commit a92c699 into ome:master Nov 28, 2019
@sbesson sbesson deleted the omero_ext_path branch November 28, 2019 13:15
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.

3 participants