Bug 503 - PV should allow the user to manually enter a sample count
Summary: PV should allow the user to manually enter a sample count
Status: CONFIRMED
Alias: None
Product: PulseView
Classification: Unclassified
Component: Acquisition (show other bugs)
Version: unreleased development snapshot
Hardware: All All
: Normal normal
Target Milestone: ---
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-11-29 14:21 CET by Uwe Hermann
Modified: 2014-11-29 14:21 CET (History)
0 users



Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Uwe Hermann 2014-11-29 14:21:33 CET
PV currently shows a 1-2-5 based list of "number of samples to acquire" in a dropdown. The list is limited to a certain number of samples when the LA reports that (e.g. 8MByte for the ChronoVu LA8), which is correct.

However, PulseView only gives the user the choice of a few pre-selected number of samples to be acquired. Example for the LA8:

100
200
500
...
1M
2M
5M
8388608

Thus, the user is not able to select, say, 6M or 7M or any other number of samples that is not in that 1-2-5 list.

The dropdown should also be a user-editable field (at the same time) though to give the user a chance to select arbitrary numbers (based on the use-case).

Making the widget editable is pretty trivial, but probably needs a bit more code to properly handle this. Here's a start if someone wants to work on this:


--- a/pv/widgets/sweeptimingwidget.cpp
+++ b/pv/widgets/sweeptimingwidget.cpp
@@ -53,6 +53,8 @@ SweepTimingWidget::SweepTimingWidget(const char *suffix,
        connect(&value_, SIGNAL(editingFinished()),
                this, SIGNAL(value_changed()));

+       list_.setEditable(true);