Skip to content

Bugfix#11

Open
kantic wants to merge 2 commits intomicrosoft:masterfrom
kantic:bugfix
Open

Bugfix#11
kantic wants to merge 2 commits intomicrosoft:masterfrom
kantic:bugfix

Conversation

@kantic
Copy link

@kantic kantic commented Apr 6, 2020

Suggested improvements:

  • Rename some of the wav files to have a uniform naming

  • strip the elements of the config parameter noise_types_excluded in order to allow comma separated list with spaces between the elements

  • read the config parameter total_snrlevels as int in order to avoid exception in the function np.linspace(...)

kantic added 2 commits April 6, 2020 16:04
- renames noise_test/Neighbor*.wav to noise_test/NeighborSpeaking*.wav to conform to the naming in noise_train directory
- renames noise_train/AirportAnnouncement_11.wav to AirportAnnouncements_11.wav to conform to the other AirportAnnouncement noise files
- the config value total_snrlevels is read as int instead of float in order to avoid rutime error in np.linspace(...)
- strips the list of strings which is read from the config value noise_types_excluded in order to allow comma separated lists with spaces
  (as suggested in the documentation of the config file)
@msftclas
Copy link

msftclas commented Apr 6, 2020

CLA assistant check
All CLA requirements met.

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