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

fix(api): check virtual smoothie before copying udev file to robot #1960

Merged
merged 1 commit into from
Jul 30, 2018

Conversation

sanni-t
Copy link
Member

@sanni-t sanni-t commented Jul 30, 2018

overview

Adds a check for virtual smoothie before copying udev files to user_storage on server start

changelog

server/main.py

review requests

@sanni-t sanni-t requested a review from btmorr July 30, 2018 17:26
@codecov
Copy link

codecov bot commented Jul 30, 2018

Codecov Report

Merging #1960 into edge will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             edge    #1960   +/-   ##
=======================================
  Coverage   32.23%   32.23%           
=======================================
  Files         428      428           
  Lines        6819     6819           
=======================================
  Hits         2198     2198           
  Misses       4621     4621

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 37c6f63...dcef2cf. Read the comment docs.

Copy link
Contributor

@b-cooper b-cooper left a comment

Choose a reason for hiding this comment

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

👍 works for me

@sanni-t sanni-t force-pushed the api_server-udev-vsmoothie-fix branch from c6c6b54 to dcef2cf Compare July 30, 2018 18:14
@sanni-t
Copy link
Member Author

sanni-t commented Jul 30, 2018

Made correction as pointed out by Ben

@sanni-t sanni-t requested a review from Kadee80 July 30, 2018 18:19
Copy link
Contributor

@btmorr btmorr left a comment

Choose a reason for hiding this comment

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

🆗

@sanni-t sanni-t merged commit 9a31f3d into edge Jul 30, 2018
@mcous mcous deleted the api_server-udev-vsmoothie-fix branch August 6, 2018 18:14
b-cooper added a commit that referenced this pull request Aug 6, 2018
* feat(protocol-designer): make pipettes eagerly drop tips (#1946)

- drop tips at the end of each step unless the next step for a pipette uses change tip: "never"

Closes #1706

* fix(docker): Switch out dumb-init, add modules tools & udev config (#1952)

- Switch to Resin's default init system
- add avrdude apk & avrdude config file
- add udev rules for modules
Closes #1822

* feat(comp): Add IntervalWrapper to interaction enhancers (#1942)

* fix(protocol-designer): fix styling of pause and mix step items (#1948)

Closes #1947

* feat(protocol-designer): add help link to PD nav (#1945)

- add 'help-circle' icon
- create OutsideLinkButton component for nav
- remove negative margin from navbar .bottom
- update PD knowledge base URLs

Closes #1941

*  feat(protocol-designer): add i18next and start with tooltips (#1949)

Add the i18next library to help organize our copy, and start off by moving tooltip copy into
translations json file.

* feat(protocol-designer): refactor and restyle LabwareSelectionModal (#1929)

* LabwareSelectionModal component matches design (was AddLabwareModal)
* factor out shadows and borders to complib css vars
* fix IngredientsList; cleanup borders
* make IngredIndividual "hoverable" (presentation only)

*  fix(api): maintain speed state after collision recovery (#1953)

* fix(api): Explicitely set gantry speed after testing an axis for skipping

* removes calls to self.set_speed() from driver

* fix(api): check virtual smoothie before copying udev file on server start (#1960)

* feat(app): Add continuous polling to modules during run (#1961)

* feat(api): Add clear method to RPC SessionManager (#1969)

* feat(protocol-designer): refactor and restyle timeline terminal items (#1967)

- fix bug with `robotStateTimeline` selector creating extra "steps" in the timeline for eager drop tip, introduced in #1706 
- fix related bug where substeps didn't generate for `changeTip: "never"` steps
- fix bug where hovering over Mix highlights wells of all labware, not just the relevant labware
- renamed a handful of selectors touched by this PR to `getSpam` instead of `spam`
- create TerminalItem component
- change copy of text in initial deck setup, final deck state, and corresponding headers
- remove flask icon from initial deck setup
- remove `StepTypeWithEnd` and remove `any` typing that was a workaround for it
- remove `deckSetupMode` selector
- update all places timeline index is used to no longer add 1 to the index to skip over deck setup step (timeline warnings, timeline errors, liquids on deck at step in timeline, well selection)

Closes #1930 and closes #1974
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