Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ac coupling not working as intended #1885

Open
AuraBF opened this issue Jan 24, 2025 · 2 comments
Open

Ac coupling not working as intended #1885

AuraBF opened this issue Jan 24, 2025 · 2 comments
Labels

Comments

@AuraBF
Copy link

AuraBF commented Jan 24, 2025

Environment:

  • OS: Windows 10
  • FW Version: V0.33
  • SW Version: V2.0.0

Describe the bug
The issue i'm describing is related to the software ac coupling. The main problem is that when you set the vertical cursor of the trace other than 0 (by double tapping), the cursor will move in relation to the vertical voltage knob, which isn't really the intended purpose of the function.
I'm not sure if this problem is present in 1.5 or earlier versions.

Expected behavior
The ideal behavior would be to have the vertical trace fixed on the relative position of the graph instead of the voltage.

Screenshots
(the screenshots are pretty self explanatory, changing the vertical scale changes the position of the trace which isn't ideal)

Image
Image

If further info is required, i'll provide it.

@AuraBF AuraBF added the bug label Jan 24, 2025
@AuraBF AuraBF changed the title <Instrument>: <Title> <Instrument>: <Title> Ac coupling not working as intended Jan 24, 2025
@AuraBF AuraBF changed the title <Instrument>: <Title> Ac coupling not working as intended Ac coupling not working as intended Jan 24, 2025
@AlexandraTrifan
Copy link
Contributor

Hi,

The plot channel handle is directly linked to Vertical Position knob. But the position on the plot can only be described as relative to the Vertical Volts/Division value. So when modifying the Volts/Dev value from 10mV to 3mV, the handle will maintain its value of 13mV and visually move on the plot.
Please correct me if I am wrong - from what I understand, the issue here is the following: the Vertical Position should automatically change its value from around 13mV to around 4mV when the Volts/Div is changed from 10mV to 3mV, such that the signal would maintain its position in the 4th vertical division of the plot?

-Alexandra

@AuraBF
Copy link
Author

AuraBF commented Jan 31, 2025

The way i see it and how it works with some oscilloscopes (I'm not certain it works like this for all of them) is that when ac coupling mode is applied, the vertical position will not move when changing the volt/div knob. And this makes sense since we have completely eliminated the dc offset.

The vertical position in the current case acts like the DC part of the signal when in fact the signal shouldn't have any dc in it, and in turn it moves the signal with the volts/div knob.

This could be implemented in a way that when ac coupling is enabled, the vertical position (In volts) is grayed to indicate it being disabled and use a different variable which indicates the relative position of the trace in relation to the graph (10 divisions in total so a number from +5 to -5 float), and when maybe you change from ac to dc coupling, you convert that relative value to the one corresponding to the position in volts like before.

I also want to add that the "enhancement" label might be more appropriate for this issue.

Please correct me if have said any nonsense since without a scope in my hands right now I can't verify the things i indicated above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants