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

polarimeter_converter is missing from conditions #15

Closed
sparksn opened this issue Mar 3, 2016 · 5 comments
Closed

polarimeter_converter is missing from conditions #15

sparksn opened this issue Mar 3, 2016 · 5 comments

Comments

@sparksn
Copy link

sparksn commented Mar 3, 2016

polarimeter_converter is not showing up in the conditions list any more.
It was there for most runs before run 10644.

@sdobbs
Copy link
Collaborator

sdobbs commented Mar 3, 2016

The DAQ version might need to be updated?

On Thu, Mar 3, 2016 at 2:00 PM Nathan A. Sparks notifications@github.com
wrote:

polarimeter_converter is not showing up in the conditions list any more.
It was there for most runs before run 10644.


Reply to this email directly or view it on GitHub
#15.

@DraTeots
Copy link
Collaborator

DraTeots commented Mar 3, 2016

Updated the daq. Looks it is there. Question, would it be a good idea now to run update_epics for previous runs? (Assuming it would use start_time and set polarimeter_converter correctly?)

@sdobbs
Copy link
Collaborator

sdobbs commented Mar 3, 2016

I actually have a slightly different code to run over previous runs,
because for that one has to access everything through MYA commands.
update_epics mostly takes the "current" values. I'm not sure of the best
way to merge the codes. MYA commands are not quite foolproof...

On Thu, Mar 3, 2016 at 5:02 PM Dmitry Romanov notifications@github.com
wrote:

Updated the daq. Looks it is there. Question, would it be a good idea now
to run update_epics for previous runs? (Assuming it would use start_time
and set polarimeter_converter correctly?)


Reply to this email directly or view it on GitHub
#15 (comment).

@sdobbs
Copy link
Collaborator

sdobbs commented Mar 4, 2016

Old runs fixed.

On Thu, Mar 3, 2016 at 5:07 PM Sean Dobbs seandobbs@gmail.com wrote:

I actually have a slightly different code to run over previous runs,
because for that one has to access everything through MYA commands.
update_epics mostly takes the "current" values. I'm not sure of the best
way to merge the codes. MYA commands are not quite foolproof...

On Thu, Mar 3, 2016 at 5:02 PM Dmitry Romanov notifications@github.com
wrote:

Updated the daq. Looks it is there. Question, would it be a good idea now
to run update_epics for previous runs? (Assuming it would use start_time
and set polarimeter_converter correctly?)


Reply to this email directly or view it on GitHub
#15 (comment).

@sdobbs
Copy link
Collaborator

sdobbs commented Mar 29, 2016

polarimeter_converter is showing up for new runs. Probably can close this issue.

@sparksn sparksn closed this as completed Mar 29, 2016
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

No branches or pull requests

3 participants