ISISComputingGroup / ISISComputingGroup/IBEX

Oscillating collimator: check and turn motor on before moving

Aperta
#7,085 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
3
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

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.