-
Notifications
You must be signed in to change notification settings - Fork 238
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
ScanR: fix exposure time units #1187
Conversation
Tried uploading the file on Trout, with the following Insight client The import goes through and thumbnails are generated, but the images are not visible on insight (under the dataset). And there is an exception in the import log as well,
|
I would be very surprised if that's at all related to this PR. @joshmoore (or anyone else)? |
I'd guess that the processor wasn't running, but I don't know why that would be the case.... Looking at the logs:
which is something @manics and I have certainly seen before. Definitely not related. |
The ome-xml data looks fine with the patch applied. Please merge. |
ScanR: fix exposure time units
--rebased-to #1198 |
See http://www.openmicroscopy.org/community/viewtopic.php?f=13&t=7554.
To test, verify that exposure times are smaller with this change. I was using
data/from_skyking/scanr/nadine
to test, where the original exposure time was50.0
and the new time is0.05
. Other datasets should produce similar results.