Skip to content
This repository has been archived by the owner on Oct 26, 2022. It is now read-only.

Operating parameters don't actually change #2

Closed
ebokhour opened this issue Mar 4, 2016 · 5 comments
Closed

Operating parameters don't actually change #2

ebokhour opened this issue Mar 4, 2016 · 5 comments

Comments

@ebokhour
Copy link
Contributor

ebokhour commented Mar 4, 2016

When changing operating parameters using the python tool, it outputs a listing which correctly shows the new values. However, if I immediately read the housekeeping, it shows the default values, as if the change was never made. This seems to be true of other parameters, too. I confirmed that no change is made in the hardware by measuring current in the trim heater dummy load after changing the current to 100 mA, or to 227 mA. This may be due to a very recent change, since yesterday we did observe current in the trim heater dummy load.

@xcthulhu
Copy link
Member

xcthulhu commented Mar 6, 2016

I'm looking into it.

In the future, please provide a script to reproduce the bug...

TESSUser added a commit that referenced this issue Mar 7, 2016
@xcthulhu xcthulhu closed this as completed Mar 7, 2016
@ebokhour
Copy link
Contributor Author

ebokhour commented Mar 7, 2016

Verified OK.

@TESSUser TESSUser reopened this Mar 7, 2016
@TESSUser
Copy link
Contributor

TESSUser commented Mar 7, 2016

After updating caladan with git pull origin master, the bug is still present. The attached script should run but fails. It did work, briefly, before the issue was closed this morning.

`#!/bin/bash
#This is a script used to reproduce the bug in TESScience/tessfpe Issue #2.

set -euo pipefail
DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )"
PYTHON=${DIR}/python

load_wrapper 6.1t.5
operating_parameters --set heater_1_current 100
house_keeping --channels heater_1_current`

@TESSUser
Copy link
Contributor

TESSUser commented Mar 7, 2016

I realize I'm running these shell scripts from the tessfpe/scripts directory, and most of your shell scripts call the Python scripts located in $DIR/python. Is that the problem?

@TESSUser
Copy link
Contributor

TESSUser commented Mar 7, 2016

Needed to Make the install again. Works now.

@TESSUser TESSUser closed this as completed Mar 7, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants