Skip to content

Amr fix - #104

Open
jesalinaphan wants to merge 2 commits into
masterfrom
amr-fix
Open

Amr fix#104
jesalinaphan wants to merge 2 commits into
masterfrom
amr-fix

Conversation

@jesalinaphan

@jesalinaphan jesalinaphan commented May 13, 2026

Copy link
Copy Markdown
Collaborator

Remove popup from AMR gui when running measurement. Resolves #100

@ElPsyKurisu ElPsyKurisu left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall very nice, but please check the comments as we need to fix the virtual_keithley and ideally get a parity match between the AMR gui and IV gui

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Im assuming the live_plot= false turns off the second window popping up?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think it would be nice to have the same pause or stop capability of AMR, also by chance could we make the returned data more than just a flat line, assume you have like a resistor or something connected so it shows like a line or something

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So we should not have virtual_specific instrument. it should always be virtual_generic, aka virtual_sourcemeter in this case. So if possible could you rename it and make sure it follows the same conventions as the other virtual_instruments?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

another thing i dont think it should inherit from scpi, since we can technically have awgs that are not scpi

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AMR Gui bug

2 participants