Eric Wieser

Add to PyPI

4

The name `pyuvc` [is unused on PyPI](https://pypi.python.org/pypi?%3Aaction=search&term=pyuvc&submit=search) - it would be great if you could publish the package there so that this module becomes `pip install`able

There is already a package on PyPI, [`monotonic`](https://pypi.python.org/pypi/monotonic), that backports the `time.monotonic` addition of python 3.6 - it would make this package more lightweight to use it!

Various fixes

2

I can split these up if any are objectionable Last commit is the most valuable - it allows quotes in default values, descriptions, and enum info

This is possibly by design, but it is somewhat frustrating. At present, it prints out the nodelet manager, which is less useful.

`eval` is used, which is unsafe

5

In the following places: - https://github.com/ros-perception/calibration/blob/20d3bbf5ec78027831f75cf90c2aac737a00e978/calibration_estimation/src/calibration_estimation/single_transform.py#L48 - https://github.com/ros-perception/calibration/blob/20d3bbf5ec78027831f75cf90c2aac737a00e978/calibration_estimation/src/calibration_estimation/joint_chain.py#L52 - https://github.com/ros-perception/calibration/blob/20d3bbf5ec78027831f75cf90c2aac737a00e978/calibration_estimation/src/calibration_estimation/urdf_params.py#L144 What type of input is this trying to parse?

DAQmxFunctions.array_type requires 'WRITABLE' flag even for `const float *`

1

the `pointer_type_array` regex should be split in two, into `pointer_type_const_array` and `pointer_type_array`

pep8-ify function names

5

Following on from [this comment](https://github.com/clade/PyDAQmx/issues/30#issuecomment-233812704). So instead of ``` python task.CfgSampClkTiming( source=None, rate=1, activeEdge=daq.Val_Rising, sampleMode=daq.Val_FiniteSamps, sampsPerChan=1 ) ``` it would become ``` python task.cfg_sample_clk_timing( source=None, rate=1, active_edge=PyDAQmx.VAL_RISING, sample_mode=PyDAQmx.VAL_FINITE_SAMPS, samps_per_chan=1 )...

Remove DAQmx_ prefix from module attributes

3

That way instead of doing this and causing arbitrary namespace pollution: ``` python from PyDAQmx import * DAQmx_Some_Function(DAQmx_SomeConstant) ``` We can use the safer ``` python import PyDAQmx PyDAQmx.Some_Function(PyDAQmx.SomeConstant) ```

Allows functions to to take a `const DualMC33926MotorShield&` argument, and be able to read currents without controlling motors