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

feat(api): add script to run threshold-finding alg on existing pressure sensor data #14549

Merged
merged 12 commits into from
Mar 12, 2024

Conversation

caila-marashaj
Copy link
Contributor

@caila-marashaj caila-marashaj commented Feb 26, 2024

This is a testing script that allows us to read in existing csv data, and use it to try out algorithms we might want to implement in firmware for detecting when a pipette tip first touches water. Afterward, we can use metadata in the provided csv to compare results with "true" height, and determine the efficacy of each one.

The csv data being used here is currently an instance of final_report.csv, which conglomerates the time, pressure sensor output, plunger positions, and z stage positions of many individual trials of liquid probing.

This addresses EXEC-138.

@caila-marashaj caila-marashaj changed the title add script feat(api): add script to run threshold-finding alg on existing pressure sensor data Mar 7, 2024
Copy link

codecov bot commented Mar 11, 2024

Codecov Report

Attention: Patch coverage is 0% with 184 lines in your changes are missing coverage. Please review.

Project coverage is 67.53%. Comparing base (dc381e8) to head (4b64ba9).
Report is 79 commits behind head on edge.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             edge   #14549      +/-   ##
==========================================
- Coverage   67.79%   67.53%   -0.26%     
==========================================
  Files        2516     2511       -5     
  Lines       72117    72391     +274     
  Branches     9280     9280              
==========================================
  Hits        48889    48889              
- Misses      21010    21284     +274     
  Partials     2218     2218              
Flag Coverage Δ
hardware 56.04% <0.00%> (-1.77%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
...ware/opentrons_hardware/scripts/lld_data_script.py 0.00% <0.00%> (ø)

... and 8 files with indirect coverage changes

@ryanthecoder ryanthecoder marked this pull request as ready for review March 11, 2024 16:31
@ryanthecoder ryanthecoder requested a review from a team as a code owner March 11, 2024 16:31
Copy link
Member

@sfoster1 sfoster1 left a comment

Choose a reason for hiding this comment

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

make those averagers classes please

hardware/opentrons_hardware/scripts/lld_data_script.py Outdated Show resolved Hide resolved
hardware/opentrons_hardware/scripts/lld_data_script.py Outdated Show resolved Hide resolved
hardware/opentrons_hardware/scripts/lld_data_script.py Outdated Show resolved Hide resolved
Copy link
Member

@sfoster1 sfoster1 left a comment

Choose a reason for hiding this comment

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

lgtm! thank you!

@ryanthecoder ryanthecoder merged commit c824269 into edge Mar 12, 2024
14 of 15 checks passed
Carlos-fernandez pushed a commit that referenced this pull request May 20, 2024
…re sensor data (#14549)

This is a testing script that allows us to read in existing csv data,
and use it to try out algorithms we might want to implement in firmware
for detecting when a pipette tip first touches water. Afterward, we can
use metadata in the provided csv to compare results with "true" height,
and determine the efficacy of each one.

The csv data being used here is currently an instance of
`final_report.csv`, which conglomerates the time, pressure sensor
output, plunger positions, and z stage positions of many individual
trials of liquid probing.

This addresses
[EXEC-138](https://opentrons.atlassian.net/browse/EXEC-138?atlOrigin=eyJpIjoiMDQxMmVjMjI4ZGY5NGU3NDk2NDEzZjRiODEyZThjOTEiLCJwIjoiaiJ9).

[EXEC-138]:
https://opentrons.atlassian.net/browse/EXEC-138?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ

---------

Co-authored-by: Ryan howard <[email protected]>
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