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

707 add visr b01 1 as a beamline with devices #708

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

stan-dot
Copy link
Contributor

Fixes #707

Instructions to reviewer on how to test:

  1. make sure dodal connect b-01-1 is successful
  2. panda and manta should connect (skipping the stage for now)

Checks for reviewer

  • Would the PR title make sense to a scientist on a set of release notes
  • If a new device has been added does it follow the standards
  • If changing the API for a pre-existing device, ensure that any beamlines using this device have updated their Bluesky plans accordingly
  • Have the connection tests for the relevant beamline(s) been run via dodal connect ${BEAMLINE}

@stan-dot stan-dot linked an issue Jul 26, 2024 that may be closed by this pull request
Copy link

codecov bot commented Jul 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.93%. Comparing base (69bd320) to head (7a544ba).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #708      +/-   ##
==========================================
+ Coverage   94.90%   94.93%   +0.02%     
==========================================
  Files         115      116       +1     
  Lines        4656     4675      +19     
==========================================
+ Hits         4419     4438      +19     
  Misses        237      237              

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

@stan-dot stan-dot self-assigned this Jul 26, 2024
@stan-dot stan-dot force-pushed the 707-add-visr-b01-1-as-a-beamline-with-devices branch from 64e281b to 9ec6d16 Compare August 16, 2024 10:02
@stan-dot
Copy link
Contributor Author

this is still not connecting :(

@stan-dot stan-dot marked this pull request as ready for review August 16, 2024 15:38
@stan-dot stan-dot force-pushed the 707-add-visr-b01-1-as-a-beamline-with-devices branch 3 times, most recently from 6df8028 to 6b6c6fd Compare August 23, 2024 07:15
src/dodal/beamlines/b01-1.py Outdated Show resolved Hide resolved
Comment on lines 17 to 20
static_directory_provider = StaticDirectoryProvider("/tmp/bluesky_test_static")
# set_directory_provider(PandASubdirectoryProvider())

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
static_directory_provider = StaticDirectoryProvider("/tmp/bluesky_test_static")
# set_directory_provider(PandASubdirectoryProvider())
set_directory_provider(StaticDirectoryProvider("/dls/b01-1/data/"))

File system seems to be hiccuping (again) but when it's recovered find a sensible visit dir for this beamline

Copy link
Contributor Author

Choose a reason for hiding this comment

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

is the file system gaslighting me at the moment?

image

Copy link
Contributor

Choose a reason for hiding this comment

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

I think someone needs to raise a ticket because it's been consistently like that for days.

src/dodal/beamlines/b01-1.py Outdated Show resolved Hide resolved
# set_directory_provider(PandASubdirectoryProvider())


def panda(
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a panda on this beamline? I don't see it on the beamline synoptic screen, nor is there a soft-ioc for it in kubernetes

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, there is. there is aan argocd migration going on at the moment.

@marcelldls mentioned it shouldn't take too long

src/dodal/beamlines/b01-1.py Outdated Show resolved Hide resolved
src/dodal/beamlines/b01-1.py Outdated Show resolved Hide resolved
return device_instantiation(
AravisDetector,
"manta",
"-DI-DCAM-02:",
Copy link
Contributor

Choose a reason for hiding this comment

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

this device is disconnected on the synoptic for me, but -DI-DCAM-03: is a Manta that is present?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it should be and Marcell is doing the IOCs into argocd migration this week afaik

@DiamondJoseph
Copy link
Contributor

Realised I was looking at BL01B: is VISR 01B or 01C? There's no synoptic screen for the latter.

@stan-dot
Copy link
Contributor Author

@DiamondJoseph , ViSR is the C variant and it does not have an epics screen

@stan-dot stan-dot force-pushed the 707-add-visr-b01-1-as-a-beamline-with-devices branch from 5456f80 to 5adfca4 Compare August 28, 2024 08:58
@stan-dot stan-dot force-pushed the 707-add-visr-b01-1-as-a-beamline-with-devices branch 2 times, most recently from c73d3e7 to 27a7af2 Compare September 13, 2024 08:27
@stan-dot
Copy link
Contributor Author

panda is supposed to be off at the moment

@stan-dot stan-dot force-pushed the 707-add-visr-b01-1-as-a-beamline-with-devices branch from c4d47b2 to 17592bc Compare September 13, 2024 13:28
@stan-dot
Copy link
Contributor Author

trying to fix manta connect errors, pending comment from @marcelldls

@stan-dot stan-dot force-pushed the 707-add-visr-b01-1-as-a-beamline-with-devices branch from 17592bc to 7a544ba Compare September 13, 2024 16:43
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.

Add ViSR (b01-1) as a beamline with devices
3 participants