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

Plate fix #839

Merged
merged 7 commits into from Mar 13, 2013
Merged

Plate fix #839

merged 7 commits into from Mar 13, 2013

Conversation

jburel
Copy link
Member

@jburel jburel commented Mar 11, 2013

@gusferguson
Copy link

@jburel

Testing using OMERO.insight-4.4.6-527-0f1af2e-dirty-ice33-b180 on Mac 10.8 user-6 read-annotate-1 on howe

Behaves as expected

  • no crash when clicking on empty well
  • permissions appropriate

Remaining issues:

  • name discrepancy - still some name discrepancy, but seems to vary from plate to plate - see attached screenshots

Well names

@jburel
Copy link
Member Author

jburel commented Mar 12, 2013

There is a difference between the name of the plate and the name of the image.
What you see in the header of the central panel is the name of the image

@jburel
Copy link
Member Author

jburel commented Mar 12, 2013

Before we go to far, we need to decide what we want to display. The indication [96-Well plate] is added to the display and not part of the name. This was requested by users.
We also have a ticket for editing the well's name see https://trac.openmicroscopy.org.uk/ome/ticket/10307

@jburel
Copy link
Member Author

jburel commented Mar 12, 2013

Following discussion with @gusferguson: fine as it is.
We need to have "real" HCS users feedback.

@gusferguson
Copy link

Agreed - leave as is and try and get some plate user's feedback.

joshmoore added a commit that referenced this pull request Mar 13, 2013
@joshmoore joshmoore merged commit 4fef8f6 into ome:dev_4_4 Mar 13, 2013
@jburel jburel mentioned this pull request Mar 13, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants