Skip to content

Trajectory scanning plan#64

Draft
Relm-Arrowny wants to merge 9 commits into
mainfrom
p99-trajectory-scanning
Draft

Trajectory scanning plan#64
Relm-Arrowny wants to merge 9 commits into
mainfrom
p99-trajectory-scanning

Conversation

@Relm-Arrowny

Copy link
Copy Markdown
Collaborator

Fixes #26

Instructions to reviewer on how to test:

  1. Do thing x
  2. Confirm thing y happens

Checks for reviewer

  • Would the PR title make sense to a user on a set of release notes

@codecov

codecov Bot commented Jun 17, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 34.78261% with 30 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.35%. Comparing base (a08213d) to head (138b6a4).

Files with missing lines Patch % Lines
...uesky/beamlines/p99/plans/sample_stage_scanning.py 34.78% 30 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##              main      #64      +/-   ##
===========================================
- Coverage   100.00%   96.35%   -3.65%     
===========================================
  Files           30       31       +1     
  Lines          777      823      +46     
===========================================
+ Hits           777      793      +16     
- Misses           0       30      +30     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Relm-Arrowny

Copy link
Copy Markdown
Collaborator Author

Tested it the plan run but, I cannot seem to trigger detector.

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.

Hard ware trajectories scan on p99.

2 participants