M-Labs/thermostat
9
0
Fork 5
Code Issues 9 Pull Requests 5 Releases Wiki Activity

pytec regression get_pwm() #31

New Issue
Closed
opened 2020-12-20 02:45:03 +08:00 by jbqubit · 4 comments
jbqubit commented 2020-12-20 02:45:03 +08:00
Contributor
Copy Link
tec = Client(host="192.168.1.52")
tec.get_pwm()

/home/britton/miniconda3/envs/thermostat37/bin/python3.7 /home/britton/gitlab/brittonlab/artiq-lab/thermostat/mytest.py
Traceback (most recent call last):
  File "/home/britton/gitlab/brittonlab/artiq-lab/thermostat/mytest.py", line 78, in <module>
    tec.get_pwm()
  File "/home/britton/miniconda3/envs/thermostat37/lib/python3.7/site-packages/pytec-0.0-py3.7.egg/pytec/client.py", line 58, in get_pwm
  File "/home/britton/miniconda3/envs/thermostat37/lib/python3.7/site-packages/pytec-0.0-py3.7.egg/pytec/client.py", line 37, in _get_conf
TypeError: string indices must be integers
``` tec = Client(host="192.168.1.52") tec.get_pwm() ``` ``` /home/britton/miniconda3/envs/thermostat37/bin/python3.7 /home/britton/gitlab/brittonlab/artiq-lab/thermostat/mytest.py Traceback (most recent call last): File "/home/britton/gitlab/brittonlab/artiq-lab/thermostat/mytest.py", line 78, in <module> tec.get_pwm() File "/home/britton/miniconda3/envs/thermostat37/lib/python3.7/site-packages/pytec-0.0-py3.7.egg/pytec/client.py", line 58, in get_pwm File "/home/britton/miniconda3/envs/thermostat37/lib/python3.7/site-packages/pytec-0.0-py3.7.egg/pytec/client.py", line 37, in _get_conf TypeError: string indices must be integers ```
astro commented 2020-12-21 03:55:44 +08:00
Copy Link

Do you run the latest firmware?

The response to pwm is now a one-line JSON array instead of two lines as before.

Do you run the latest firmware? The response to `pwm` is now a one-line JSON array instead of two lines as before.
jbqubit commented 2020-12-21 04:24:50 +08:00
Author
Contributor
Copy Link

I was using latest firmware... I built it.

I'm not my test bench at the moment. Did you update pytec to ingest the new JSON?

I was using latest firmware... I built it. I'm not my test bench at the moment. Did you update pytec to ingest the new JSON?
astro commented 2020-12-22 08:08:42 +08:00
Copy Link

Yes, pytec has been updated.

Yes, pytec has been updated.
sb10q commented 2020-12-28 17:49:09 +08:00
Owner
Copy Link

So I guess we can close this. @jbqubit please reopen if this is still relevant.

So I guess we can close this. @jbqubit please reopen if this is still relevant.
sb10q closed this issue 2020-12-28 17:49:12 +08:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: M-Labs/thermostat#31
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?