acc issueshttps://gitlab.triumf.ca/hla/acc/-/issues2017-09-25T11:37:48-07:00https://gitlab.triumf.ca/hla/acc/-/issues/5Bender position calculation in xml2optr/optr2xml2017-09-25T11:37:48-07:00Carla BarquestBender position calculation in xml2optr/optr2xmlFound an offset before and after bends in emma.xml when running the following command on hlx:
```
/home/cbarquest/acc/lib/xml2optr.py '/home/cbarquest/acc/isac/path/emma.xml' -E 79.3591 -q 7 -m 33533.80 -Q 0 -t '/home/cbarquest/acc/isac...Found an offset before and after bends in emma.xml when running the following command on hlx:
```
/home/cbarquest/acc/lib/xml2optr.py '/home/cbarquest/acc/isac/path/emma.xml' -E 79.3591 -q 7 -m 33533.80 -Q 0 -t '/home/cbarquest/acc/isac/tune/emma.xml' -o '/home/cbarquest/acc/lib/emmatest/'
```
The output I put for the first version in beam/envelope highlights the drift that I had to manually change for the generated xml2optr sy.f file to match the reference sy.f output:
https://gitlab.triumf.ca/hla/beam/envelope/blob/c3c71a2e9bc9afb68587dfec0f7f05f7c0dc74b6/optr/isac/emma/sy.f#L37
Drifts before and after the bends were off by +/-26.18cm. Note the sequence emma.xml file was originally generated by optr2xml(I think?) so not sure if the position calculation is off in optr2xml or xml2optr.
Here are the relevant emma.xml path/sequence/tune files:
* [Path emma.xml](https://gitlab.triumf.ca/hla/acc/blob/7398886db772256ccd284a540305f2dec6564e9a/isac/path/emma.xml)
* [Sequence emma.xml](https://gitlab.triumf.ca/hla/acc/blob/7398886db772256ccd284a540305f2dec6564e9a/isac/sequence/emma.xml)
* [Tune emma.xml](https://gitlab.triumf.ca/hla/acc/blob/7398886db772256ccd284a540305f2dec6564e9a/isac/tune/emma.xml)
And here are the reference sy.f and data.dat files:
* [Reference emma sy.f](https://gitlab.triumf.ca/beamphys/transoptr/blob/55f7ee22a74be564ad1dfc4cd810aadba9b2746c/example/ISAC/EMMA/sy.f)
* [Reference emma data.dat](https://gitlab.triumf.ca/beamphys/transoptr/blob/55f7ee22a74be564ad1dfc4cd810aadba9b2746c/example/ISAC/EMMA/data.dat)Thomas PlancheThomas Planchehttps://gitlab.triumf.ca/hla/acc/-/issues/4Stange attributes of ao in sequence XML files2017-09-12T15:17:08-07:00Thomas PlancheStange attributes of ao in sequence XML filesThis is only a suggestion:
I see things like:
<ao ccb='IMS:CCB6:VOL' EGU='V' DRVL='0.0' DRVH='1000.0'>IMS:YCB8:VOL</ao>
The common power supply should be a separated ao. I mean it should look instead like:
<ao EGU='V' DRVL='0.0' DRVH='1...This is only a suggestion:
I see things like:
<ao ccb='IMS:CCB6:VOL' EGU='V' DRVL='0.0' DRVH='1000.0'>IMS:YCB8:VOL</ao>
The common power supply should be a separated ao. I mean it should look instead like:
<ao EGU='V' DRVL='0.0' DRVH='1000.0'>IMS:YCB8:VOL</ao>
<ao EGU='V' DRVL='0.0' DRVH='1000.0'>MS:CCB6:VOL</ao>
What do you think?
I also see things like:
<ao rw='RO' type='scan'>IMS:RPM24:SUBRPM</ao>
what is rw??Spencer KiySpencer Kiyhttps://gitlab.triumf.ca/hla/acc/-/issues/3change 'at' to 'center'2017-08-15T14:49:41-07:00Thomas Planchechange 'at' to 'center'The tag 'at' is creating confusion as to whether it corresponds to the location of the element entrance or center (with respect to the beginning of the sequence). It does corresponds the the location of the element center. It was propose...The tag 'at' is creating confusion as to whether it corresponds to the location of the element entrance or center (with respect to the beginning of the sequence). It does corresponds the the location of the element center. It was proposed during the last HLA meeting to rename it to 'center' to avoid further confusion.Thomas PlancheThomas Planchehttps://gitlab.triumf.ca/hla/acc/-/issues/2Add accurate DRVL and DRVH to every ao2017-07-27T12:10:11-07:00Thomas PlancheAdd accurate DRVL and DRVH to every aoOne could probably write a script to do that: get from EPICS the value of DRVL and DRHV for every valid PV, and append them to the sequence XMLs.
I need the DRVL and DRVH for transoptr. For now I sneaked them in the tune file. but they d...One could probably write a script to do that: get from EPICS the value of DRVL and DRHV for every valid PV, and append them to the sequence XMLs.
I need the DRVL and DRVH for transoptr. For now I sneaked them in the tune file. but they do not belong there.Spencer KiySpencer Kiy