OpenHantek / openhantek

OpenHantek is a DSO software for Hantek (Voltcraft/Darkwire/Protek/Acetech) USB digital signal oscilloscopes
http://openhantek.org/
GNU General Public License v3.0
771 stars 199 forks source link

Broken voltage levels #158

Closed phmarek closed 6 years ago

phmarek commented 6 years ago

With eccfd9641fcbc4d25f5740a97d940a4ac10722bb, I get good voltage measurements.

With 33723056459adec972927aa46331bd5675b02886 and the same settings (2V/div, DC), the displayed waveform seems to have only half the voltage.

I don't know whether the voltage divider settings sent to the oscilloscope are wrong, or whether it's just the display that's broken, though.

davidgraeff commented 6 years ago

Which DSO model do you use? The code base was modernized and restructured recently and I can only test with one of the supported types. There might be something messed up.

phmarek commented 6 years ago

I'm using a DSO2250. Thanks for the quick answer!

phmarek commented 6 years ago

Any updates? Still happens with c66be8a976a6af63706f692d638254bdb7e6f272 for me.

Looks like the relay settings and/or offsets sent to the oscilloscope or so are broken - small changes in the zero-line setting make the graph jump to the upper resp. lower border.

davidgraeff commented 6 years ago

The problem is, at the moment there are mainly contributions from people with a dso6022. That model doesn't support offset and relays. So nobody of the current contributors can test the offset and relay code.

davidgraeff commented 6 years ago

The openhantek2 branch contains a self-calibration and adjustable gain dividers, this should help.

uunicorn commented 6 years ago

I have the same problem on DSO2250. openhantek2 seems to be even worse - I can't seem to change the time base. Also offset/trigger markers are jumping up and down when trying to drag them around.

phmarek commented 6 years ago

That is on the master branch .... any chance for a fix on the openhantek2 branch as well?

uunicorn commented 6 years ago

openhantek2 looks pretty broken to me. I'm building against qt 5.11 with gcc-5.4. Got a couple of compilation errors:

After fixing those errors I was able to run the application, but it's still quite broken:

Nevertheless, the fix from the linked PR seems to resolve the problem with cmdGetLimits. If you want, I can raise the PR for openhantek2 with the same set of changes as in #241 , but I can't really test it without properly fixing all the other issues I've listed above.

davidgraeff commented 6 years ago

The OpenHantek2 branch was meant to restructure the codebase to increase maintainability and add devices more easily. But there are some issues as commented. it was only tested with one compiler and one scope at hand.