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.

Sign up for GitHub

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

Jump to bottom

Fix CI #5

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Fix CI #5

wants to merge 3 commits into from

Conversation

mrakitin
Copy link
Member

No description provided.

@mrakitin mrakitin requested review from jklynch and AbbyGi March 24, 2021 15:52
@mrakitin
Copy link
Member Author

Looks like there is an interactive prompt waiting for a user input, so maybe we need a fix similar to NSLS-II-PDF/profile_collection#24 (commit NSLS-II-PDF/profile_collection@f02293a).

@jklynch
Copy link
Contributor

jklynch commented Apr 16, 2021 •
edited
Loading

This line seems to say there should be a "beamtime" object created like this:

bt=_start_beamtime(<PI_last>, <saf_num>,<experimenter_list>, wavelength=<wavelength_num>)

but I don't think we can just do that.

I think there may be a configuration file that should provide this information.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jklynch jklynch jklynch approved these changes

@AbbyGi AbbyGi Awaiting requested review from AbbyGi

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mrakitin @jklynch