ISISComputingGroup / ISISComputingGroup/IBEX
Oscillating collimator: check and turn motor on before moving
- Lingua principale
- Nessun dato sulla lingua
- Stelle
- 6
- Fork
- 2
- Merge medio
- 16h 40m
- PR unite (30g)
- 2
Descrizione
to `c:\Instrument\Apps\EPICS\support\galil-old\master\gmc\galil_oscillating_collimator.gmc` (note: `galil-old`) in the `#OSCIL` step need to turn motor on. Could change `BG~a` to `SH~a;BG~a` but I know we often do `SH~a;WT100;BG~a` but i am not sure we want to introduce a delay. The motor should already be on, so in case a rapid SH;BG can potentially cause an issue maybe safest to add something like this to the top of `#OSCIL`:
```
IF (_MO~a = 1)
SH~a; WT100
ENDIF
```
In addition, it does not e.g. check stop `_SC~a` after a move, or limit switch status e.g. `_LR~a` and `_LF~a` or have a `#LIMSWI` routine like in `galil_default_footer`. May be worth putting setting an error variable in controller if `_SC` shows move doesn't complete and then exposing this variable to DB
## acceptance criteria
- patched on LET
- merged into master
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Direzione di ricerca
Inizia da galil_oscillating_collimator.gmc, concentrandoti sul passaggio #OSCIL, quindi confronta la gestione del motore e dei finecorsa con la routine #LIMSWI in galil_default_footer. Convalida la modifica su LET, incluso il comportamento descritto del motore e dello stato del movimento, e conferma che il risultato sia stato unito in master.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Ambito
- embedded-iot
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100