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

Audio output is inconsistent #88

Closed
christophkern opened this issue Jul 14, 2018 · 2 comments

Comments

Projects
None yet
2 participants
@christophkern
Copy link

commented Jul 14, 2018

The audio output is supposed to give a tone for each measured spectrum indicating the current overhead column density. It's a useful feature when the operator is not able to look at the screen. However, the current audio output is a bit confusing. I suggest changing to the following scheme:

  1. Make audio output optional
  2. If audio output is turned on, use a constant volume. This way, the user can set it to a desired level.
  3. Instead of the volume, scale the pitch of the tone to the SO2 column density
  4. Don't use a different type of tone for negative values. These are common and not really a problem.
  5. Do use a different type of tone if an error occurs, for example if the GPS is lost or if the spectrometer fails or if the current spectrum is identical to the last one (indicating a problem with the spectrometer).
  6. Not sure how practical, but it could be cool to have a voice indicate whenever pre-determined column amounts are exceeded or values drop below them. E.g. say '1,000' if the last column was less than 1,000 ppmm but the current one is larger than 1,000 ppmm or vice versa. There may have to be a minimum time between voice messages to avoid issues if values persist around one of the thresholds. Thresholds could either be set by the user or we could try the following: 200, 500, 1000, 2000, 5000, 8000, 10,000, I know these are not evenly spaced, but let's try for numbers that are only 3 syllables when spoken.
@dnorgaard-usgs

This comment has been minimized.

Copy link
Contributor

commented Jul 17, 2018

It appears not all audio devices support change in pitch: https://docs.microsoft.com/en-us/windows/desktop/multimedia/changing-pitch-and-playback-rate. My Windows 10 laptop does not support it. For now, I will do a check to see if pitch change is supported. If it is, I will set the pitch. If it is not, I will set the volume.

Added option for audio on spectrometer configuration dialog:

image

Made modifications to use same sound for negative values as the positive ones. Different sound is already in use for GPS error.

@dnorgaard-usgs

This comment has been minimized.

Copy link
Contributor

commented Jul 17, 2018

The PlaySound function used in the code can play a sound specified by the given file name, resource, or system event. Probably we would have to create wav files for each of those thresholds and install them on the user's PC somewhere for this to work. Will keep thinking on it but at the moment it is not looking very practical.

https://msdn.microsoft.com/en-us/library/windows/desktop/dd743680(v=vs.85).aspx

dnorgaard-usgs added a commit to dnorgaard-usgs/MobileDOAS that referenced this issue Jul 17, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.