Difference between revisions of "TODO"

From sigrok
Jump to navigation Jump to search
m
 
(75 intermediate revisions by 10 users not shown)
Line 3: Line 3:
=== Hardware support ===
=== Hardware support ===


* <s>Support for analog devices, starting with oscilloscopes.</s>
* Clean up device-specific globals in hardware drivers, to properly permit multiple devices per driver: fx2lafw, zeroplus-logic-cube, others(?)
* <s>Add a Hantek DSO-2090 driver.</s>
* Chronovu LA8/LA16 driver: support multiple devices properly (currently only one is supported).
* <s>Add a tool for firmware extraction of Hantek DSO-2xxx.</s>
** <s>Should be written in Python.</s>
** <s>Should parse the .sys files (http://www.csn.ul.ie/~caolan/publink/winresdump/winresdump/doc/pefile.html).</s>
* libudev:
** Make libudev optional, <s>only build MSO driver if it's available</s>. If the user enables the MSO driver, but libudev is not available, politely print an error message and exit.
** Have OLS driver use libudev if available
** Find alternative to libudev for Windows and Mac OS X.
* Clean up device-specific globals in hardware drivers, to properly permit multiple devices per driver: <s>asix-sigma</s>, <s>demo</s>, <s>link-mso19</s>, <s>openbench-logic-sniffer</s>, fx2lafw, zeroplus-logic-cube
* ZEROPLUS driver:
** Fix implementation to work like the other sigrok drivers wrt acquisition/session handling ([[User:Uwe Hermann|Uwe Hermann]]).
** handle multiple ZP devices properly: currently all devices share the same device context struct
** move sr_usb_dev_inst so sr_dev_inst, not device context struct. Currently only the last discovered device has the right USB bus/address, all other discovered devices are just lost.
* Asix SIGMA driver:
** support multiple devices properly (currently only one is supported).
* Chronovu-LA8 driver:
** support multiple devices properly (currently only one is supported).
* In order to support the CLI --run option, devices will need to provide their buffer size to the frontend, with a DI option.
* add "mode=sine" parameter to demo driver, to have it output a sine wave in SR_DF_ANALOG floats
* agilent-dmm:
** factor out serial_readline2 to common/serial.c version
** do away with baud rates in profile?
** time-limited and continuous mode
* fluke-dmm:
** time-limited and continuous mode


=== Input file formats ===
=== Input modules ===


* Add CSV input format support.
* Add OLS input module support.
* Add VCD input format support.
* Add gnuplot input module support ([[User:Uwe Hermann|Uwe Hermann]]).
* Add OLS input format support.
* Add gnuplot input format support ([[User:Uwe Hermann|Uwe Hermann]]).
* Add Scanalogic input format support ([[User:Uwe Hermann|Uwe Hermann]]).
 
=== Output file formats ===
 
* <s>Basic analog text output: display as float, unit suffix?</s>
* All output modules should use GString API instead of estimating output size.
** binary
** csv
** chronovu_la8
** gnuplot
** ols
** text/ascii
** text/bits
** text/hex
** vcd
* Hex output: Not all lines/columns are of the same length (as they are with real LAs).
* VCD: All samplenumbers are 0 (instead of the proper samplecount).
* OLS, gnuplot: Hangs on MinGW for some reasons (not sure if related).
* CSV: Check if this really works and the syntax is correct. It has various trailing commas, it seems, which should probably be removed.
* Add Scanalogic output format support ([[User:Uwe Hermann|Uwe Hermann]]).
* Add output for latex package [http://www.ctan.org/tex-archive/graphics/pgf/contrib/tikz-timing tikz-timing].
* Add output for browser waveform viewer/editor written in javascript/JSON called [http://code.google.com/p/wavedrom WaveDrom].
* <s>Have a look at DrawTiming http://drawtiming.sourceforge.net (ImageMagick frontend commandline tool).</s>
 
=== New features ===
 
* Factor out libsigrok filter into modular transform system.
** take arguments in the regular thing:key=value format
** integrate current probe compression filter
** resampling module: sample up/down based on factor argument
** noise filter: filter out pulses shorter than the samplerate's period
** software triggering
*** integrate current basic trigger functionality from saleae driver
*** for streaming devices, opportunity for more complex triggers than the hardware can do
*** We will have an extra datafeed type SR_DF_LOGIC_INDEXED, which contains [samplenumber, sample] instead of just raw samples. Samplenumber is a uint64_t, and sample is unchanged (unitsize). Frontends must be able to handle both, and modular filters in libsigrok or the sampling filter in libsigrokdecode may convert from SR_DF_LOGIC to SR_DF_LOGIC_INDEXED at any time.
 
=== Windows port ===
 
* Implement anykey.c replacement.
* Fix/finish serial port handling code, that's not working, yet.
* Fix/workaround libusb 1.0 [[Windows]] port issues:
** Device renumeration not yet supported (needed for FX2 based LAs).
** File descriptor / socket based polling not supported in Windows. Workaround (short-term): Use a thread in sigrok.


=== Miscellaneous ===
=== Miscellaneous ===


* Use glib functions for file handling, not manual fopen/fclose etc., for portability reasons.
* Check all public API calls to see whether they need an sr_context parameter.
* Factor out input/output file formats to libsigrokfile.
* All direct uses of file descriptors in the API should disappear, and GIOChannel should be used instead (portability issues).
* Use const for all parameters which are not changed by the API call.
* There must not be any lib-internal data structures that a frontend can manipulate directly. Internal stuff must be non-accessible, except via our defined API calls.
* <s>[http://developer.apple.com/library/mac/#documentation/Darwin/Reference/ManPages/man3/tcsetattr.3.html Mac OS X: B460800] is not available it seems. Find the correct header (if any) and/or find a (possibly platform-specific) workaround, or drop this on Mac OS X.</s>
* Run the code receiving the samples as quickly as possible, run output format code and other non-timing-critical stuff (protocol decoders) in an extra thread. The problem is nicely visible with FX2-based logic analyzers, the longer an output format takes to produce its output, the better your chances to get "oops, I just lost samples" errors.
* Change all code to start counting probes at 0, not 1.
* Get rid of the use of manual polling of FDs, which is not portable (doesn't work on [[Windows]]). Use g_poll() instead (which spawns a new thread on Windows).
* sr_errno:
* sr_errno:
** Most functions should return SR_OK or SR_ERR_* (same for SRD_OK etc).
** Most functions should return SR_OK or SR_ERR_* (same for SRD_OK etc).
** Most backend functions return status as an integer, which is SR_OK if all went well, or SR_ERR_* and similar if an error occurred.
** Most backend functions return status as an integer, which is SR_OK if all went well, or SR_ERR_* if an error occurred.
** However, there is no way to pass any information back as to what went wrong &mdash; and this is important for user-friendliness.
** However, there is no way to pass any information back as to what went wrong &mdash; and this is important for user-friendliness.
** Perhaps an error code is not enough; maybe something like sigrok_errno(errorcode, "unsupported device") is better.
** Perhaps an error code is not enough; maybe something like sigrok_errno(errorcode, "unsupported device") is better.
* Add good Doxygen documentation for (at least) all public functions.
* Support specifying the firmware directory at runtime.
 
=== Before release ===
 
* <s>avoid ABI breakage in the HWCAP enum</s> (done for all enums. Documentation added in HACKING)
** <s>assign a number to the first of every section/category in the enum, with 10K or so in between assignments.</s>


== libsigrokdecode ==
== libsigrokdecode ==
==== Major features ====
* Add high-precision timer to runtc
* Add annotation mime types
* Add annotation metadata (type/value/unit) to allow clients to format the value as the user desires (ASCII/dec/hex/bin/whatever)


* Performance: ATM data passed to decode() creates lots of Python objects (-> performance, huge memory usage); find a way to make that more efficient. Use 'bytes' and/or 'memoryview' and the new Python 3 buffer API (?)
==== Other ====
* Test/implement '''config''' PD method etc.
* Annotations need an additional 'type' field, so that PDs/GUIs can differentiate between different output "things".
** Types are not yet defined, but could be things like DEFAULT, ERR, WARN, DBG, and others.
** Best handled with a keyword arg to put(), which defaults to srd.ANN_DEFAULT.
* Processing raw logic vs. everything else up the stack: (optionally) deliver pre-sampled by clock or rate? <blockquote>Yes, the controller will deliver a clock- or rate-sampled feed instead, if the PD requests this via a configuration option.</blockquote>
* <s>How to pass (and what exactly, and in which format) samples and already decoded protocol packets:
** From C to Python (i.e. from libsigrokdecode/controller to the lowest-level decoder)
** From Python to Python (i.e., from one decoder to the next in the stack)
** From Python to C (i.e., from a decoder back to libsigrokdecode, which then hands it over to the CLI/GUI, which write it to file or display it in the GUI).</s>
* <s>There will also have to be a type for "sections", i.e. grouped sets of fields that have something in common -- an EDID block, ethernet frame, etc.
** Actually not needed: these annotations will cover a larger sample range than its individual fields, so it's up to the frontend to do something visually interesting with this.</s>
* Annotations: replace string with optional list, containing annotations in decreasing length?
** quick and easy form: <tt>put(ss, es, opid_ann, "something")</tt>
** multiple strings: <tt>put(ss, es, opid_ann, ["something", "else", "yet"])</tt>
** ...or annotations of decreasing length: <tt>put(ss, es, opid_ann, ["something", "sth", "s"])</tt>
* Test/implement support for multiple outputs of one PD, and stacking different additional PDs on those outputs.
* Test/implement support for multiple outputs of one PD, and stacking different additional PDs on those outputs.
* Test/implement multiple inputs support.
* Test/implement multiple inputs support.
* <s>srd: Python implementation details should be removed from the API.</s>
* Python exception handling in libsrd: add full traceback support with srd_dbg().


== sigrok-cli ==
== sigrok-cli ==
* Support for printing/selecting '''one or more''' specific annotation types for output.
* <s>enforce input/output options in PDs: don't send logic if PD only accepts i2c etc.</s> figure out what to do with inputs[] and outputs[] (1-wire case)
* Python exception handling in libsrd: add full traceback support with srd_dbg()
* Make specification of triggering capabilities more detailed, so that frontends/GUIs can auto-generate GUI forms out of that metadata.
* Add a --run option, which just starts sampling with the supplied or default samplerate. Contrary to --time or --samples you don't need to explicitly specify a duration or number of samples here, that's automatically determined by the available memory of the logic analyzer and the selected samplerate. Devices that can stream continuously will do so, others will just fill a buffer and then stop.
** TODO: Similar to --continuous? Merge both?
* sigrokdecode probe name matching on level 0 decoders: if the probe names supplied on the cmdline for acquisition or supplied from .sr file match the probes a level 0 PD requires '''exactly''', fill in the probemap accordingly. In all other cases, throw up an error if a level 0 PD has required probes but none were specified as arguments.
** (what about optional probes?)


== fx2lafw ==
== fx2lafw ==
Line 144: Line 42:
* Frequency/pulse counter
* Frequency/pulse counter
* Signal generator
* Signal generator
* Fix power consumption during enumeration.
** Non-LP FX2's consume more than 100mA during enumeration. This is outside of the USB specification
** Official workaround is to enumerate first as full-speed, then as high-speed. This is overkill for sigrok.
** Investigate putting the CPU to sleep (except for processing interrupts) until enumerated.


== pulseview ==
== PulseView ==
*<strike>Add signal naming</strike>
*Add save [Working, but held up by changes needed to sigrok-cli and libsigrok]
**Add progress reporting
*Migrate to C++11?
*Make it responsive to SIGABRT etc.
*Add signal decoding
*Add signal arrangement
*Add signal overview
*Add triggering
*Support analog
*Threaded sampling - use Lars-Peter Clausen's asynchronous stuff.
*Proper error reporting instead of qDebug messages
**When file-open fails
**When the LA capture fails
*Selection/cursors
*Signal groups
*Paint speed profiling information.
*Code documentation
*Transition coding
*Support different open file formats
*Tidy up ruler
**Add some graphical refinements.
**Stop numbers colliding at high zoom
* Have some way to configure driver parameters. i.e. serial port for RS-232 devices.
* Have some way to configure libsigrok* debug level
*Store config with QSettings
 
== sigrok-qt ==
 
* ...
 
== sigrok-gtk ==
 
* File->Open: sigrok-gtk should remember the directory of the last file that was opened/loaded, so that you don't have to browse to the same directory again and again if you load multiple files.
* sigrok-gtk should remember the last window position/size upon shutdown/restart.
* A manpage needs to be written for sigrok-gtk ([[User:Uwe Hermann|Uwe Hermann]]).
* sigrok-gtk needs a time scale (work in progress, haavares on IRC).
 
== Project ==
 
* Update the relevant wiki API pages with high-level info (details will be in the Doxygen-docs): [[Protocol decoder API]], [[Formats and structures]], [[Hardware plugin API]], [[Input API]], [[Output API]].
* Make a user-friendly protocol list/lookup page, based on a data-only backend.
** Integrate these into the list: http://en.wikipedia.org/wiki/Category:Serial_buses.
* Protocol decoder repository management:
** manifest file in every PD directory, containing:
*** list of files to install
*** list of test dumps in sigrok-dumps
*** list of output files to match test dump runs
** to check before accepting PD into repository:
*** make sure sigrok-dumps has the test files, and the output matches
*** run PD against memory profiler, make sure it doesn't go over X
*** check code coverage againsst test files, make sure it's 100%
*** run against pep8 with custom filter list
** parse TODO items out of PDs, and present them on the website
* FreeBSD packages/ports.
 
== Subprojects ==
* Python bindings for libsigrok and libsigrokdecode.
== Mugshots ==
We need better mugshots for the following devices:
* [[Acute PKLA-1216]] (cropped corner, lens distortion, orange hue)
* [[Braintechnology USB Interface V2.x]] (orange hue)
* [[ChronoVu LA8]] (orange hue, severe lens distortion, uneven lighting)
* [[Microchip PICkit2]] (Color temperature too low; cannot fix from existing png, uneven lighting)
* [[Pico Technology PicoScope 2203]]
* [[Pico Technology PicoScope 2205]]
* [[RDing TEMPer1K2]] (none of the angles are usable as a 'pretty' mugshot)
* [[Voltcraft DL-120TH]] (Bottom shadow makes it impossible to correctly identify the edge)
* [[Voltcraft DL-140TH]]
* [[Robomotic BugLogic 3]] (Poor contrast)
* [[MiniLA Mockup]] (severe lens distortion)
* [[QuantAsylum QA100]] (none of the angles are usable as a 'pretty' mugshot)
* [[Rigol VS5202D]] (none of the angles are usable as a 'pretty' mugshot)
* [[V&A VA18B]] (mugshot blurry due to excessive perspective and color correction)
 


=== A few notes on taking good pictures ===
* Deduplicate property setting functionality between SamplingBar and DeviceOptions - make SamplingBar a properties container
The idea behind mugshots is that they will be placed on a transparent background in the [[Supported hardware]] page. It's surprising how small imperfections in an image become stringently obvious when used as such.
* Analog interpolation (cosine? sinc? some kind of interpolation filter?)
These guidelines presented below are just that, guidelines. If you have a better way to take mugshots (and there are many better ways), feel free to use your method.
* Signal overview
* Use a matte (non-reflectve), neutral background; usually white is the best choice. A piece of paper or a poster-board works wonders. Do not use a colored background, as light tends to reflect from the background, giving the device edges a color hue. This hue becomes stringently obvious when the mugshot is placed on a transparent background.
* Transition coding
* Use midday light on a clear day, but avoid sunlight hitting the device directly to prevent reflections. This is by far the easiest setup to work with.
** Alternatively, use 5000K to 6500K CFLs. Do not use incandescent bulbs or 3000K CFLs.
* Avoid using the built-in camera flash. Any front flash can easily create unwanted reflections on the device.
* Set your camera to the highest resolution.
* Set your camera to record image metadata. Metadata is very useful in applying lens correction filters, most notably, reducing distortion.
* If your lens exhibits severe lens distortion, consider taking the shot from further back, and/or dial down the zoom until you get a more acceptable result. Although the resulting image will have "more background" and the cropped image will be of a lower resolution, a lower resolution image still provides better clarity than a higher resolution image with aggressive lens correction applied in post-processing.
* If your camera supports it, take pictures in RAW format. Images in RAW format are much less prone to artifacts when applying color corrections.
* If available, place an 18% gray card somewhere in the frame. This helps establish correct color balance. This is helpful only if shooting in RAW mode.
* If lighting conditions permit, set your camera to ISO100 and aperture between F8 and F11. For most lenses, this provides the sharpest picture.


<gallery>
== sigrok-meter ==
File:NI_GPIB-USB-HS.png|<small> Excellent picture, with good contrast, no obvious lens distortion, no color hue, and no halo from cropping the background</small>
File:Tp4000zc_front.png|<small> Acceptable picture, but with uneven lighting. The lower right and upper left corners recieve more light than the rest of the device. This is a result of poor choice in positioning the lights.</small>
File:Braintechnology_usb_interface_v2x.png|<small> Color hue from non-neutral background is visible after background is cropped out
File:Minila_mockup.jpg|<small> Severe lens distortion</small>
File:Mastech_va18b_device_front.png|<small> Blurry image due to too agressive post-processing corrections</small>
File:Robomotic_buglogic3.jpg|<small> Poor lighting required the iso sensitivity to be set too high, resulting in poor contrast. Color balance is also off.</small>
<gallery>


Contact [mailto:spam@fil.ter mrnuke] on #sigrok for more details.
* Make the list of measurements look nicer without using too much additional space.
* Add graphs.
** [http://www.pyqtgraph.org/ PyQtGraph] looks like a nice graphing library.
*** Works with PyQt and PySide.
*** Pure python, easy to install.
*** Few dependencies: '''scipy''' is not required any more in the upcoming version 0.9.9, '''numpy''' is already a dependency of the python bindings.
*** From looking at the examples, it seems more than fast enough for our requirements.
** Pass all samples (not only the most recent one) to the GUI thread and store them there.
*** The packet payload object is currently not thread save.

Latest revision as of 13:11, 19 July 2020

libsigrok

Hardware support

  • Clean up device-specific globals in hardware drivers, to properly permit multiple devices per driver: fx2lafw, zeroplus-logic-cube, others(?)
  • Chronovu LA8/LA16 driver: support multiple devices properly (currently only one is supported).

Input modules

  • Add OLS input module support.
  • Add gnuplot input module support (Uwe Hermann).

Miscellaneous

  • Check all public API calls to see whether they need an sr_context parameter.
  • sr_errno:
    • Most functions should return SR_OK or SR_ERR_* (same for SRD_OK etc).
    • Most backend functions return status as an integer, which is SR_OK if all went well, or SR_ERR_* if an error occurred.
    • However, there is no way to pass any information back as to what went wrong — and this is important for user-friendliness.
    • Perhaps an error code is not enough; maybe something like sigrok_errno(errorcode, "unsupported device") is better.
  • Support specifying the firmware directory at runtime.

libsigrokdecode

Major features

  • Add high-precision timer to runtc
  • Add annotation mime types
  • Add annotation metadata (type/value/unit) to allow clients to format the value as the user desires (ASCII/dec/hex/bin/whatever)

Other

  • Test/implement support for multiple outputs of one PD, and stacking different additional PDs on those outputs.
  • Test/implement multiple inputs support.
  • Python exception handling in libsrd: add full traceback support with srd_dbg().

sigrok-cli

fx2lafw

  • Analog sampling: 1-channel, 2-channel, voltage meter
  • Trigger pin
  • Clock input
  • Clock output
  • Frequency/pulse counter
  • Signal generator
  • Fix power consumption during enumeration.
    • Non-LP FX2's consume more than 100mA during enumeration. This is outside of the USB specification
    • Official workaround is to enumerate first as full-speed, then as high-speed. This is overkill for sigrok.
    • Investigate putting the CPU to sleep (except for processing interrupts) until enumerated.

PulseView

  • Deduplicate property setting functionality between SamplingBar and DeviceOptions - make SamplingBar a properties container
  • Analog interpolation (cosine? sinc? some kind of interpolation filter?)
  • Signal overview
  • Transition coding

sigrok-meter

  • Make the list of measurements look nicer without using too much additional space.
  • Add graphs.
    • PyQtGraph looks like a nice graphing library.
      • Works with PyQt and PySide.
      • Pure python, easy to install.
      • Few dependencies: scipy is not required any more in the upcoming version 0.9.9, numpy is already a dependency of the python bindings.
      • From looking at the examples, it seems more than fast enough for our requirements.
    • Pass all samples (not only the most recent one) to the GUI thread and store them there.
      • The packet payload object is currently not thread save.
Retrieved from "https://sigrok.org/w/index.php?title=TODO&oldid=15296"