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

sandbox: Sound level meter implementation #153

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

fsheikh
Copy link
Contributor

@fsheikh fsheikh commented Jan 15, 2019

Sound level meter (SLM) is a typical application for embedded system working on sound input. SLM or sound pressure meter as it is sometimes called is specified by IEC and ANSI S1.4 standards. Interestingly while there are many Matlab based implementations, upon need I could not find one in native code. I was surprised since Liquid SDR has all the building blocks available and thus an app can be rapidly built and it worked!

I hope this will be a useful application for embedded users of Liquid. At the very least it provides yet another example program for reference to Liquid users. Please check out the top of source file for references, usage and testing options for this program.

Summary: Sandboxed sound level meter application
for liquid SDR. Implements ANSI S1.4 standard for
sound level meter with octave one-third filterbanks.

Signed-off-by: Faheem Sheikh <[email protected]>
@JayKickliter
Copy link

clang's scan-build reports this minor issue on your new file:

ccc-analyzer -g -O2  -msse4.1 -Wall -fPIC -I. -Iinclude  -c -o sandbox/sound_level_meter.o sandbox/sound_level_meter.c
sandbox/sound_level_meter.c:89:39: warning: format specifies type 'size_t' (aka 'unsigned long') but the argument has type 'uint64_t'
      (aka 'unsigned long long') [-Wformat]
                            filename, samples_read);
                                      ^~~~~~~~~~~~
1 warning generated.

@fsheikh
Copy link
Contributor Author

fsheikh commented Jan 16, 2019

clang's scan-build reports this minor issue on your new file:

Thanks for having a look and pointing this out, I will fix this along with a couple other typos I have noticed.

Summary: One issue pointed out in review and couple of
typos fixed.

Signed-off-by: Faheem Sheikh <[email protected]>
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.

2 participants