Skip to content

Add script to run acquisition with fastcs-odin#70

Merged
GDYendell merged 1 commit intomainfrom
acquisition-script
Dec 1, 2025
Merged

Add script to run acquisition with fastcs-odin#70
GDYendell merged 1 commit intomainfrom
acquisition-script

Conversation

@GDYendell
Copy link
Contributor

@GDYendell GDYendell commented Nov 4, 2025

Instructions to run:

  • Create /scratch/data
  • podman run --rm -it --security-opt label=disable -v /dev/shm:/dev/shm -v /scratch/data:/data --net=host ghcr.io/diamondlightsource/eiger-detector-runtime:1.17.0
  • Run fastcs-odin IOC on main
  • Run fastcs-eiger IOC on this branch
  • Run acquisition script
  • Check /scratch/data for files

You may need to change the PVs in the launch.json and script to avoid clashes

@codecov
Copy link

codecov bot commented Nov 4, 2025

Codecov Report

❌ Patch coverage is 87.75510% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.19%. Comparing base (ff09321) to head (c357675).

Files with missing lines Patch % Lines
src/fastcs_eiger/__main__.py 0.00% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #70      +/-   ##
==========================================
+ Coverage   83.10%   84.19%   +1.09%     
==========================================
  Files          10       13       +3     
  Lines         367      405      +38     
==========================================
+ Hits          305      341      +36     
- Misses         62       64       +2     

☔ View full report in Codecov by Sentry.
📢 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.

@GDYendell GDYendell force-pushed the acquisition-script branch 4 times, most recently from c82d3df to 4a953df Compare November 18, 2025 16:29
@GDYendell GDYendell requested review from jsouter and shihab-dls and removed request for jsouter November 18, 2025 16:30
@GDYendell GDYendell force-pushed the acquisition-script branch 2 times, most recently from cdfa6e5 to 2b80b9b Compare November 28, 2025 12:05
@GDYendell GDYendell linked an issue Nov 28, 2025 that may be closed by this pull request
@GDYendell GDYendell merged commit dac89d7 into main Dec 1, 2025
36 checks passed
@GDYendell GDYendell deleted the acquisition-script branch December 1, 2025 16:13
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.

Write a script to poke PVs and run a test acquisition

1 participant