morituri icon indicating copy to clipboard operation
morituri copied to clipboard

don't keep short HTOA's if their peak level is low

Open RecursiveForest opened this issue 11 years ago • 18 comments

Right now morituri will blindly rip PREGAPs before the first track, even if they're silent. This simple patch lets morituri rip the PREGAP, then checks the rip's peak level as reported by cdparanoia to see whether it's below "SILENT" or not (default to 1e-10). If it's silent, it quietly removes the evidence from the log and cue and carries on with the rip, otherwise the original bevahiour is kept.

RecursiveForest avatar May 17 '14 18:05 RecursiveForest

isn't the data stored as integers, in which case you should safely be able to make SILENT=0?

kevmitch avatar May 31 '14 12:05 kevmitch

What’s the status of this?

ArchangeGabriel avatar Mar 11 '15 13:03 ArchangeGabriel

I'm also interested in better HTOA handling.

salixh5 avatar Mar 12 '15 18:03 salixh5

@thomasvs Could you please state here what’s wrong with this PR? Thanks a lot. :)

ArchangeGabriel avatar Mar 20 '15 00:03 ArchangeGabriel

hi! any chance that this PR will be accepted? a simple yes or no is enough! :smile: anyways, thanks for developing morituri!

dogo77 avatar Apr 06 '15 18:04 dogo77

Something like this will. It needs some interrupted time from me for testing, and I want to make sure it does this only when the silence is absolute - i.e. 100% identical to zero.

thomasvs avatar Apr 06 '15 22:04 thomasvs

Is there anything one could possibly do to help with testing?

salixh5 avatar Apr 18 '15 01:04 salixh5

+1 to this idea from me

Whovian9369 avatar Jul 21 '15 02:07 Whovian9369

@thomasvs have you taken time to look at and test this PR?

Eriner avatar Sep 06 '15 19:09 Eriner

On 09/06/2015 03:04 PM, Matt Hamilton wrote:

@thomasvs https://github.com/thomasvs have you taken time to look at and test this PR?

— Reply to this email directly or view it on GitHub https://github.com/thomasvs/morituri/pull/79#issuecomment-138114792.

No, not yet. I've bought an external cd rom that I think should handle HTOA (as I don't have my desktop anymore after the move) so I hope to do so soon.

T

Honeymoon's Great Wish You Were Her

savon - Saving your work to svn https://apestaart.org/thomas/trac/

thomasvs avatar Sep 06 '15 20:09 thomasvs

Great. I'll ping you again in a week or so to bother you about it :)

Eriner avatar Sep 06 '15 20:09 Eriner

@thomasvs Is there any progress?

JoeLametta avatar Sep 19 '15 08:09 JoeLametta

@thomasvs pinging you now. It's been 3 weeks :)

Eriner avatar Sep 27 '15 23:09 Eriner

ping

anarcat avatar Nov 24 '15 01:11 anarcat

@thomasvs I too would like a status update :)

Frogging101 avatar Jan 31 '16 19:01 Frogging101

@thomasvs ping

robotmaxtron avatar Apr 18 '16 01:04 robotmaxtron

I would be great if this PR was merged :D

baldurmen avatar May 19 '16 04:05 baldurmen

Bumping for great justice (+1)

Zorlin avatar Jul 18 '16 08:07 Zorlin