Skip to content

Triton: behaviour on switching to/from closed loop mode #4103

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

Closed
Tom-Willemsen opened this issue Mar 11, 2019 · 9 comments
Closed

Triton: behaviour on switching to/from closed loop mode #4103

Tom-Willemsen opened this issue Mar 11, 2019 · 9 comments

Comments

@Tom-Willemsen
Copy link
Contributor

As an LET instrument scientist I want the Triton system to behave correctly in closed loop mode.

From LET scientist:

The user wanted to warm the dilution to 0.3k so we put the value in the set temperature box and nothing happened. I found out that you then have to press a button for closed loop on. Again this should be automatic and the user should not have to see this as it was on SECI. You just want to give the temperature and that’s it. After turning closed loop on nothing happened still. It then turns out we have to tell it where to get a PID file in another tab. I did this and it finally started working. I then changed something in the blocks and found I could not change temperature again. It turns out it had now turned the closed loop off and lost the PID file so I had to go and put it all in again. It seems its like with the collimator that when you do anything in the blocks it forgets everything and you have to restart it all.

We should clarify exactly what the oxford triton software does when set to closed loop mode - I think it may reset the setpoint to the current temperature, but this needs testing.

Also test that the PID file is preserved on IOC restart - this applies to all the drivers that use ReadASCII (e.g. would also apply to Eurotherm)

@Tom-Willemsen Tom-Willemsen changed the title Triton: behaviour in closed loop mode Triton: behaviour on switching to/from closed loop mode Mar 11, 2019
@John-Holt-Tessella
Copy link
Contributor

We need to decide whether the PID file and closed loop should be autosaved or a setting in the macros I do not know which make sense here.

@Tom-Willemsen
Copy link
Contributor Author

On checking the labview driver, it ALWAYS sets closed loop mode on when any temperature is set. The IOC should also do this.

@Tom-Willemsen
Copy link
Contributor Author

Tom-Willemsen commented Mar 13, 2019

To do as part of this ticket:

  • Set PID file from configuration (make it a macro)
  • Set whether the driver should use the PID file from the config (could either make it a separate macro, or do a stringiftest-style check on whether the calibration file name is non-empty)
  • Set closed loop mode to ON on when any temperature is set Set closed loop mode to ON on and ensure it switched on correctly just before any temperature setpoint is sent

These changes would be good to get in in time for an experiment on LET 22/3/2019

@Tom-Willemsen
Copy link
Contributor Author

Tom-Willemsen commented Mar 22, 2019

Switching PID files from config: worked ok on actual hardware on LET

Turning closed loop mode on whenever temperature is set: on real triton fridge on LET, turning closed loop on/off makes the fridge reset it's temperature setpoint to the current temperature. This caused very odd behaviour with the changeset above, so I removed it again. There is a meeting scheduled with cryogenics where I'll ask what we should be doing about closed loop mode ( #3993 ). I'll put this in impeded in the meantime.

I think the difference is that the labview sends closed loop on first, then checks it's turned on, and then sends the temperature, whereas the changeset above assumed that tset and closed loop mode were independent. I will try implementing the ordering at the protocol level instead.

@Tom-Willemsen
Copy link
Contributor Author

Discussed this with cryogenics, there are several additional subtle issues:

  • While closed loop mode is on, the diagnostics from the fridge are severely limited. This is why cryogenics do not want to run tritons in closed loop mode all the time.
  • With closed loop off, a setpoint can be sent to the fridge, but it doesn't do anything. This is confusing for the scientists.
  • Setting closed loop mode to ON immediately after setting a temperature setpoint doesn't work correctly because the triton system changes TSet to match the current temperature on entering closed loop mode
  • Setting closed loop mode to ON immediately before setting a temperature setpoint, which is what the labview does, may be acceptable but needs further testing as cryogenics believe this may have contributed to a fault on WISH.

Cryogenics will be running a fridge down to low temperature over shutdown, and they will give us an opportunity to test on the real hardware and work with them to try and find the "least bad" solution to this issue. We will then need to apply this solution to both LabVIEW and IBEX drivers.

@kjwoodsISIS
Copy link
Contributor

When all these issues have been investigated and resolved, we must ensure to explain the behaviour of the Triton systems to the scientists, so that they understand what is and is not possible.

@Tom-Willemsen
Copy link
Contributor Author

Triton fridge NDRI1 will be available for us to test with Tomorrow (17/4/2019).

@Tom-Willemsen
Copy link
Contributor Author

Tested on a real Triton (NDRI1) with help from cryogenics, a sequence that seems to work ok is:

  • Send temperature setpoint
  • Wait 500ms
  • Set closed loop mode on
  • Wait 500ms
  • Send temperature setpoint again

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants