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

Dodal 846 device factory support for ophyd v1 #743

Open
wants to merge 5 commits into
base: dodal_846_update_mx_beamlines_to_use_device_factory
Choose a base branch
from

Conversation

rtuck99
Copy link
Contributor

@rtuck99 rtuck99 commented Jan 13, 2025

This is part of

This PR updates mx-bluesky to work with changes introduced in dodal PR

which adds support for ophyd v1 devices to device_factory

This PR builds on mx-bluesky PR:

@rtuck99 rtuck99 force-pushed the dodal_846_device_factory_support_for_ophyd_v1 branch from 1420d00 to bd85482 Compare January 13, 2025 14:16
Copy link

codecov bot commented Jan 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.97%. Comparing base (a16bb7f) to head (bd85482).

Additional details and impacted files
@@                                 Coverage Diff                                  @@
##           dodal_846_update_mx_beamlines_to_use_device_factory     #743   +/-   ##
====================================================================================
  Coverage                                                86.97%   86.97%           
====================================================================================
  Files                                                      101      101           
  Lines                                                     6933     6933           
====================================================================================
  Hits                                                      6030     6030           
  Misses                                                     903      903           
Components Coverage Δ
i24 SSX 72.69% <ø> (ø)
hyperion 96.53% <100.00%> (ø)
other 96.58% <ø> (ø)

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.

1 participant