Skip to content

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

Open
RecursiveForest wants to merge 6 commits into
thomasvs:masterfrom
RecursiveForest:master
Open

don't keep short HTOA's if their peak level is low#79
RecursiveForest wants to merge 6 commits into
thomasvs:masterfrom
RecursiveForest:master

Conversation

@RecursiveForest

Copy link
Copy Markdown
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
Copy Markdown

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

@ArchangeGabriel

Copy link
Copy Markdown

What’s the status of this?

@salixh5

salixh5 commented Mar 12, 2015

Copy link
Copy Markdown

I'm also interested in better HTOA handling.

@ArchangeGabriel

Copy link
Copy Markdown

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

@dogo77

dogo77 commented Apr 6, 2015

Copy link
Copy Markdown

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

@thomasvs

thomasvs commented Apr 6, 2015

Copy link
Copy Markdown
Owner

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.

@salixh5

salixh5 commented Apr 18, 2015

Copy link
Copy Markdown

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

@Whovian9369

Copy link
Copy Markdown

+1 to this idea from me

@Eriner

Eriner commented Sep 6, 2015

Copy link
Copy Markdown

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

@thomasvs

thomasvs commented Sep 6, 2015

Copy link
Copy Markdown
Owner

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

Eriner commented Sep 6, 2015

Copy link
Copy Markdown

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

@JoeLametta

Copy link
Copy Markdown
Contributor

@thomasvs Is there any progress?

@Eriner

Eriner commented Sep 27, 2015

Copy link
Copy Markdown

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

@anarcat

anarcat commented Nov 24, 2015

Copy link
Copy Markdown

ping

@Frogging101

Copy link
Copy Markdown

@thomasvs I too would like a status update :)

@robotmaxtron

Copy link
Copy Markdown

@thomasvs ping

@baldurmen

Copy link
Copy Markdown

I would be great if this PR was merged :D

@Zorlin

Zorlin commented Jul 18, 2016

Copy link
Copy Markdown

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.