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

Recommend MM 2022-09-20 #215

Merged
merged 1 commit into from
Oct 1, 2022
Merged

Recommend MM 2022-09-20 #215

merged 1 commit into from
Oct 1, 2022

Conversation

talonchandler
Copy link
Collaborator

In #141 @ieivanov requested an slightly newer version of MM so that a recent change to a device driver was included.
I tested recOrder with 2022-09-20 and all acquisition functionality is working. pycromanager did not require an upgrade.

This PR updates the docs and warning messages only.

@talonchandler talonchandler added this to the 0.2.0 milestone Oct 1, 2022
@talonchandler talonchandler linked an issue Oct 1, 2022 that may be closed by this pull request
@codecov-commenter
Copy link

codecov-commenter commented Oct 1, 2022

Codecov Report

Merging #215 (b371a76) into main (d4c1024) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main     #215   +/-   ##
=======================================
  Coverage   10.23%   10.23%           
=======================================
  Files          38       38           
  Lines        5160     5160           
=======================================
  Hits          528      528           
  Misses       4632     4632           
Impacted Files Coverage Δ
recOrder/plugin/widget/main_widget.py 0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@deprecated-napari-hub-preview-bot

Preview page for your plugin is ready here:
https://preview.napari-hub.org/mehta-lab/recOrder/215
Created: 2022-10-01T01:06:49.022722

@mattersoflight mattersoflight self-requested a review October 1, 2022 20:23
Copy link
Member

@mattersoflight mattersoflight left a comment

Choose a reason for hiding this comment

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

Thanks @talonchandler. The two builds are within a month - the chance of a hiccup is communication between MM and pycro-manager is negligible. We run into incompatibilty when the MM device adapter API or the zeromq versions change.

@ziw-liu ziw-liu added the enhancement New feature or request label Oct 1, 2022
@talonchandler talonchandler merged commit cac622d into main Oct 1, 2022
@talonchandler talonchandler deleted the mm-upgrade-09-20 branch October 1, 2022 23:58
@ieivanov
Copy link
Contributor

ieivanov commented Oct 4, 2022

Quick note - 2022-09-20 supports writing to NDTiff! That doesn't matter here, but it's a cool feature

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade pycromanager to 0.18.3
5 participants