Skip to content
This repository was archived by the owner on Feb 2, 2026. It is now read-only.

Fix a typo in agilentE4434B.py (a copy-paste artifact?)#63

Open
firegurafiku wants to merge 1 commit intopython-ivi:masterfrom
firegurafiku:fix-typo-agilentE4435B
Open

Fix a typo in agilentE4434B.py (a copy-paste artifact?)#63
firegurafiku wants to merge 1 commit intopython-ivi:masterfrom
firegurafiku:fix-typo-agilentE4435B

Conversation

@firegurafiku
Copy link
Copy Markdown

This patch fixed the problem which can be best demonstrated with the following snippet:

>>> import ivi
>>> a = ivi.agilent.agilentE4435B()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/<...>/ivi/agilent/agilentE4435B.py", line 35, in __init__
    super(agilentE4434B, self).__init__(*args, **kwargs)
NameError: name 'agilentE4434B' is not defined

That seems to be the same issue as #8 and #55. Copy & paste is a dangerous development methodology, isn't it a good idea to replace all the wrapper classed with something saner or at least add a unit test to ensure that all devices can default construct?

@firegurafiku
Copy link
Copy Markdown
Author

@alexforencich

Any feedback, please?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant