microsoft / microsoft/Qcodes

driver / Agilent E8267C and HP8133A change state on reinitialisation

Open
#1,208 3 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
459
Forks
359
Avg merge
3d 6h
Merged PRs (30d)
73

Description

See here:
https://github.com/QCoDeS/Qcodes/blob/f7b173fa88afae1248a732aeb2051a8e1c169b3c/qcodes/instrument_drivers/agilent/E8267C.py#L80-L81

And here:
https://github.com/QCoDeS/Qcodes/blob/f7b173fa88afae1248a732aeb2051a8e1c169b3c/qcodes/instrument_drivers/HP/HP8133A.py#L85-L86

As far as I understand, this is against our philosophy; we do **not** want the `__init__` of a driver to change the state of the instrument.

Contributor guide

Open the contributing guide

Research direction

Read the referenced sections of qcodes/instrument_drivers/agilent/E8267C.py and qcodes/instrument_drivers/HP/HP8133A.py first. Check which initialization operations change instrument state, then verify that reinitializing either driver leaves the instrument state unchanged while preserving required setup. No test file is named in the issue.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
50/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.