You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When trying to set the ramp down speed of a CAEN R803x channel, it responds with a command error.
From the manual I see that the command should be RDWN, while in the sourc it is RDW:
Is the RDW command in the source code a typo, or are there actually devices that use this command? If so, could I maybe simply add the RDWN command to the list?
The text was updated successfully, but these errors were encountered:
I'm afraid there is some inconsistency between the commands for different devices... Or perhaps something that we are missing. At a glance looks like my PS has more options related to ramp up / down, perhaps that is the reason for the shorter name? If this issue is widespread probably it would be best to have separate classes for each family of power supplies, but it would involve some work which I cannot do at this time. Perhaps it can be enough to do some kind of try / except logic to figure out the right command for a given device (atleast for now).
In summary: we cannot remove the RDW command, but I'm sure it possible to add the RDWN command somehow, feel free to open a PR with this!
When trying to set the ramp down speed of a CAEN R803x channel, it responds with a command error.
From the manual I see that the command should be
RDWN
, while in the sourc it isRDW
:hvps/src/hvps/commands/caen/channel.py
Line 44 in 639fa02
Attached is a screenshot from the R803x manual:
Is the
RDW
command in the source code a typo, or are there actually devices that use this command? If so, could I maybe simply add theRDWN
command to the list?The text was updated successfully, but these errors were encountered: