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

MuSR: 3He Refrigerator Investigation [TIMEBOX for investigation: 1/2D] and implementation #5959

Closed
kjwoodsISIS opened this issue Dec 10, 2020 · 10 comments
Assignees
Labels
3 fixed LabVIEW LabView expertise needed for most/all of this ticket support

Comments

@kjwoodsISIS
Copy link
Contributor

kjwoodsISIS commented Dec 10, 2020

As a developer, I would like to investigate the existing LabVIEW solution for controlling the 3He Refrigerator (a Heliox/ITC-503 based system), so that I can assess the feasibility of using it in IBEX.

Acceptance Criteria

  1. Talk to Muons to get a better understanding of their precise requirements
  2. The existing LabVIEW solution used to control the 3He Refrigerator used on MuSR is reviewed.
    • The LabVIEW solution is believed to be this: C:\LabVIEW Modules\Oxford Software\Muon Heliox
      • These are VIs written by Oxford Instruments (not written by ISIS)
  3. Make a rough assessment of the effort required to convert the existing LabVIEW solution to EPICS.
  4. Make a recommendation on how to proceed in the:
    • short-medium term (i.e. up to the end of Cycle 2020/04, which ends 26-03-2021)
    • long term (i.e. from the end of the TS-1 long shutdown, which ends 25-04-2022)
  5. Implement a solution

Notes

  1. This ticket requires someone with experience of LabVIEW.
  2. Possibilities for future running of the 3He Refrigerator include:
    1. Run existing LabVIEW solution on NDXMUSR. Create lvDCOM component to exchange data with LabVIEW.
    2. Run existing LabVIEW solution on a separate PC. Again, use lvDCOM component to exchange data with LabVIEW.
    3. There may be others ...
  3. Note from Cryogenics:
    • "Something I should also add now I’ve thought about it: The very same Heliox with ITC503 is also used on Neutron instruments (almost exclusively WISH actually) with what looks like the same Oxford Instruments VI."
  4. Note from Instrument Scientist (08-01-2021):
    • "As things are currently scheduled we’ll be using uSRi1, which is one of the Mercury-controlled Helioxes. If there were a problem with either the Heliox controlled inserts or the cryostat they use we would need to switch to the ITC503 controlled Heliox.
      To draw a footballing analogy, the Mercury controlled one is in the starting line-up but the ITC503 controlled one will be on the bench when the game starts."
@John-Holt-Tessella
Copy link
Contributor

Also worth comparing it to the work in the current driver for the heliox https://github.com/ISISComputingGroup/ibex_developers_manual/wiki/Mercury-Heliox to see if we can just modify that driver.

@davidkeymer davidkeymer added the LabVIEW LabView expertise needed for most/all of this ticket label Jan 7, 2021
@ThomasLohnert ThomasLohnert changed the title MuSR: 3He Refrigerator Investigation MuSR: 3He Refrigerator Investigation [TIMEBOX for investigation: 1/2D] and implementation Jan 7, 2021
@kjwoodsISIS kjwoodsISIS added this to the SPRINT_2021_01_07 milestone Jan 7, 2021
@kjwoodsISIS kjwoodsISIS added ready and removed proposal labels Jan 7, 2021
@davidkeymer
Copy link
Contributor

Current VI (C:\LabVIEW Modules\Oxford Software\Muon Heliox\Muon Heliox Front Panel2.vi) determined from SECI configurations.
Brief investigation of code revealed :

  • Mostly consists of configuration and management of up to four ITC503s.
  • The setup VI (C:\LabVIEW Modules\Oxford Software\Muon Heliox\MUON HELIOX SETUP DIALOG2.vi) allows the Heliox to be configured and the ITC parameters to be set.
  • Main VI contains logic to determine whether or not a recondense cycle is required.
  • The recondense VI itself (C:\LabVIEW Modules\Oxford Software\Muon Heliox\Muon Heliox Recondense2.vi) contains a reasonable amount of code.

Suggest studying the Mercury Heliox driver and Wiki page and liasing with a representative from the Cryogenics team during this work.

@DominicOram
Copy link
Contributor

DominicOram commented Jan 19, 2021

Needs discussion when we talk about cycle change. Do not pick up in the mean time

@John-Holt-Tessella
Copy link
Contributor

Plan is to spend time on ticket to do this as an EPICS driver and get rid of the labview. Make a start and see. Tom should not do this but is a good advisor.

@JamesKingWork
Copy link
Contributor

JamesKingWork commented Feb 16, 2021

In discussion with @Tom-Willemsen the following was agreed

This ticket has become to create new tickets resulting from the investigation:

The first ticket and communications are to be pulled into this sprint and the points on this ticket reduced to reflect the actual time spent.

@KathrynBaker
Copy link
Member

The tickets make some sense, but a ticket to track the moving parts is probably necessary as well, given the 6 new tickets that have been created. I've created that ticket (#6247) and assigned it to @JamesKingWork so that the responsibility can be maintained, and so that we don't lose any parts of the work before considering it complete.

@kjwoodsISIS
Copy link
Contributor Author

@JamesKingWork - does the 3 points include @davidkeymer 's initial investigation?

@JamesKingWork
Copy link
Contributor

@JamesKingWork - does the 3 points include @davidkeymer 's initial investigation?

Yes, but I have guessed David did about half a day on it, I don't know exactly how long he did spend. I did about half a day and the review, I imagine, was very quick.

@FreddieAkeroyd
Copy link
Member

Does this ticket need release notes or should it be marked "support" to avoid the release notes check error?

@John-Holt-Tessella
Copy link
Contributor

My feeling is most tickets should have release notes. They indicate to the scientists the work we are doing as well as what has changed with IBEX, both from code and configuration point of view.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 fixed LabVIEW LabView expertise needed for most/all of this ticket support
Projects
None yet
Development

No branches or pull requests

8 participants