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

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

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

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

wants to merge 6 commits into from

Conversation

RecursiveForest
Copy link
Contributor

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.

@kevmitch
Copy link

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

@ArchangeGabriel
Copy link

What’s the status of this?

@yacode11
Copy link

I'm also interested in better HTOA handling.

@ArchangeGabriel
Copy link

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

@dogo77
Copy link

dogo77 commented Apr 6, 2015

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

@thomasvs
Copy link
Owner

thomasvs commented Apr 6, 2015

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.

@yacode11
Copy link

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

@Whovian9369
Copy link

+1 to this idea from me

@Eriner
Copy link

Eriner commented Sep 6, 2015

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

@thomasvs
Copy link
Owner

thomasvs commented Sep 6, 2015

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
#79 (comment).

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/

@Eriner
Copy link

Eriner commented Sep 6, 2015

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

@JoeLametta
Copy link
Contributor

@thomasvs Is there any progress?

@Eriner
Copy link

Eriner commented Sep 27, 2015

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

@anarcat
Copy link

anarcat commented Nov 24, 2015

ping

@Frogging101
Copy link

@thomasvs I too would like a status update :)

@robotmaxtron
Copy link

@thomasvs ping

@baldurmen
Copy link

I would be great if this PR was merged :D

@Zorlin
Copy link

Zorlin commented Jul 18, 2016

Bumping for great justice (+1)

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.

None yet