-
Notifications
You must be signed in to change notification settings - Fork 301
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
Triton fixes #576
Triton fixes #576
Conversation
So change encoding such that they can be read correctly
@jenshnielsen dont i just cross checked on T12, and the channel numbering as i wrote it seems to be correct? |
Yes I see that too. But if i Ask the api for T0 I get nonsense such as (1.6e123) If I ask for T1 I get the value that matches |
This seems to match section 5.11.11 in the triton manual
|
argh i missed that I used a modified version, so I guess you are right. Sorry for that... lets have a quick chat on slack |
|
||
def _get_temp_channels(self): | ||
self.chan_temps = [] | ||
for i in range(1, 16): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should probably be range(1,17) so that all 16 channels are included. We are not using T16 at Triton 10 but anyway
Discussed with @MerlinSmiles on Slack and made some modifications based on his suggestions. The reg file is now optional and a few bugs fixed |
Author: Jens Hedegaard Nielsen <jenshnielsen@gmail.com> Triton fixes (#576)
@MerlinSmiles I think this is mainly your driver does this make sense to you?