Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incorporating changes from Canary IL project. #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

WolvesOfTheNight
Copy link

I made an updated version of the AD5933 library as part of a project I was working on. Here are the changes, if you are interested.

  • Updated readme
  • Updated comments
  • Added frequency code conversion function, freqToCode
  • Added 16-bit frequency sweep functions for explicit control of word size, if desired. The use of 32-bit words was consuming a lot of memory when I started working with multiple sweeps.
  • Added support for setting the start and increment frequencies by code or Hz (interger only) so that users can run sub-1Hz frequency steps if desired. This is especially useful when running several sweeps to build a larger spectrum. Support for setting frequency as a float was not added because you really need to understand the limitations of the frequency code when running sub-1Hz steps.
  • Adds support for averaging several tests at each data point.
  • Added setOutputVoltage
  • Added setSettlingCycles
  • Set getTemperature to return -666 in case of an error because -1 degrees C is a valid temperature, and quite possible for some uses of the device.

I ended up doing the data analysis in my own program. I though about updating the calibration function here, but I would end up breaking backward compatibility. I might be able to come up with a new process data function if there is sufficient interest.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant