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

Slidebook7: Fixes for repo tests #3828

Merged
merged 4 commits into from
May 25, 2022
Merged

Conversation

dgault
Copy link
Member

@dgault dgault commented May 20, 2022

There are a couple of fixes here for issues raised by failure of the repo tests: https://merge-ci.openmicroscopy.org/jenkins/job/BIOFORMATS-test-folder/lastFailedBuild/console

  • Exposure time is now saved as a double
  • MappedID is used during initFile
  • IsThisType function modified slightly to use its own dataLoader

To test simply ensure that the repo test for the new Slidebook7 sample turn green.

@dgault dgault added this to the 6.10.0 milestone May 20, 2022
@sbesson sbesson mentioned this pull request May 23, 2022
@dgault
Copy link
Member Author

dgault commented May 24, 2022

With this PR included the repo tests on merge-ci are now green for the new samples: https://merge-ci.openmicroscopy.org/jenkins/job/BIOFORMATS-test-folder/186025/console

@dgault dgault requested a review from sbesson May 24, 2022 09:56
@sbesson sbesson merged commit f867012 into ome:develop May 25, 2022
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.

2 participants