Update specaug.py

[aky15]

What?

Support configuration and control of the methods of applying time and freq mask, i.e. masking with 0 or masking with the mean value. Masking with 0 by default.

Why?

Helps to switch the methods of applying time and freq mask.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 54.59%. Comparing base (b0e10d7) to head (444d52a).
Report is 3 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff             @@
##           master    #5803       +/-   ##
===========================================
+ Coverage   32.16%   54.59%   +22.43%     
===========================================
  Files         773      771        -2     
  Lines       70997    70732      -265     
===========================================
+ Hits        22835    38616    +15781     
+ Misses      48162    32116    -16046     

Flag	Coverage Δ
test_integration_espnet2	?
test_python_espnet2	52.98% <ø> (?)
test_python_espnetez	?
test_utils	20.61% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[sw005320]

Thanks.
I see that you have prepared the flag.
Will you prepare the option argument to replace the value?

[aky15]

Yes, I add the default value True to make 'replace_with_zero' an option argument. @sw005320
Masking with 0 or by mean value has already been implemented in mask_along_axis function. However, the flag is missed in the SpecAug so I added it.

[sw005320]

Oh, I see.
Do you mean that the default behavior would be changed with this?

@pyf98, can you check this PR?

[pyf98]

Thanks for adding the option. @aky15 I guess it won't change the default behavior? If this is true, then it looks good.

[aky15]

Yes, it won't change the default behavior. @sw005320 @pyf98

[sw005320]

OK, thanks!