search

labscript-suite-temp-2 / labscript

The labscript Python library provides a translation from simple Python code to complex hardware instructions. The library is used to construct a "connection table" containing information about what hardware is being used and how it is interconnected. Devices described in this connection table can then have their outputs set by using a range of functions, including arbitrary ramps.
BSD 2-Clause "Simplified" License
0 stars 0 forks source link

StaticDDS class does not ask parent device for default unit conversion classes like DDS class #33

Open philipstarkey opened 7 years ago

philipstarkey commented 7 years ago

Original report (archived issue) by David Meyer (Bitbucket: dihm, GitHub: dihm).

It's a minor annoyance, but getting the right number of zeros on a Novatech to set 80MHz in Hz is tedious.

There is a simple copy/paste upgrade from the DDS class. I can send a pull request with the fix I've been using for a while now.

philipstarkey commented 7 years ago

Original comment by Chris Billington (Bitbucket: cbillington, GitHub: chrisjbillington).

I have fixed this in the Spielman labs as well, but there are some other changes there too that I need to look at before merging the lot in which is why it hasn't been merged yet. Getting the Spielman and mainline forks back together is a priority at the moment so this will happen soon.

I'd say keep using your own fix for now and I'll get our fix into the mainline soon! The fixes ought to be compatible - as you've seen it's not that complicated a problem to fix. I'll leave this bug report open until the fix is in mainline.