-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: Example 8 - calculate RMS, dBSPL and dBA levels #254
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
excellent first contribution @milanredon 😊
examples/008_calculate_levels.py
Outdated
|
||
# %% | ||
# In a loop, for each sound, create a :class:`.OverallLevel` object, set its signals, | ||
# get and the overall RMS, dBSPL and dBA levels, then Write the results into the first .csv file. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@milanredon there is a missing verb here:
"get and XXX the overall RMS"
my guess XXX = display? print?
examples/008_calculate_levels.py
Outdated
|
||
# %% | ||
# In a loop, for each sound, create a :class:`.OverallLevel` object, set its signals, | ||
# get and the overall RMS, dBSPL and dBA levels, then Write the results into the one .csv file per sound. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@milanredon there is a missing verb here:
"get and XXX the overall RMS"
my guess: XXX = plot?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@milanredon it is probably "[...] dBSPL and dBA vs time, [...]"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good one :)
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #254 +/- ##
==========================================
Coverage 100.00% 100.00%
==========================================
Files 79 81 +2
Lines 4648 4974 +326
==========================================
+ Hits 4648 4974 +326 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…s/pyansys-sound into feat/example-with-dB-levels
for more information, see https://pre-commit.ci
…figure in the doc + some minor improvements
New example to calculate overall levels, levels versus time and to write the results in csv files.