Skip to content

Conversation

@rclarke0
Copy link
Contributor

@rclarke0 rclarke0 commented May 20, 2025

Overview

Add serial number and retrieval count tracking for stackers
Added stackable PCR plates and custom lid definition to allow for protocol to run without user intervention

Test Plan and Hands on Testing

  • tested with stacker run logs
  • ran updated protocol on 2 ABR robots

Changelog

  • Added serial number and retrieval count tracking for stackers
  • Added custom stackable PCR plate and custom thermocycler disposable lid so that it can be stacked on PCR plate
  • Removed steps in 4 stacker protocol that caused a collision with the waste chute

Review requests

Risk assessment

@rclarke0 rclarke0 requested review from a team as code owners May 20, 2025 20:17
@rclarke0 rclarke0 requested review from sfoster1 and skowalski08 May 21, 2025 16:04
@codecov
Copy link

codecov bot commented May 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 25.36%. Comparing base (679bb46) to head (0a26a92).
Report is 2 commits behind head on edge.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             edge   #18402       +/-   ##
===========================================
- Coverage   57.85%   25.36%   -32.50%     
===========================================
  Files        3252     3252               
  Lines      276196   276132       -64     
  Branches    32255    32194       -61     
===========================================
- Hits       159799    70035    -89764     
- Misses     116204   206070    +89866     
+ Partials      193       27      -166     
Flag Coverage Δ
app 1.26% <ø> (-46.45%) ⬇️
protocol-designer 19.21% <ø> (ø)
step-generation 4.82% <ø> (-0.01%) ⬇️

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

see 1661 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

Nice!

@rclarke0 rclarke0 merged commit 5aaa46f into edge May 21, 2025
36 of 37 checks passed
@rclarke0 rclarke0 deleted the abr-track-stacker-moves branch May 21, 2025 16:57
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