Skip to content

Commit

Permalink
Add pre-emphasis status reporting to whipper's logfiles
Browse files Browse the repository at this point in the history
Fixes #120
  • Loading branch information
JoeLametta committed Feb 5, 2017
1 parent 6ddb5d0 commit fb271f0
Showing 1 changed file with 10 additions and 3 deletions.
13 changes: 10 additions & 3 deletions morituri/result/logger.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,8 @@ def logRip(self, ripResult, epoch):
lines = []

# Ripper version
# Only implemented in whipper (ripResult.logger)
lines.append("Log created by: whipper %s (%s logger)" % (
morituri.__version__, ripResult.logger))
lines.append("Log created by: whipper %s (internal logger)" %
morituri.__version__)

This comment has been minimized.

Copy link
@Freso

Freso Feb 5, 2017

Member

Not sure what's going on here, why this is part of the "add preemphasis" commit, but eh.

This comment has been minimized.

Copy link
@JoeLametta

JoeLametta Feb 5, 2017

Author Collaborator

It's just an unrelated cosmetic change


# Rip date
date = time.strftime("%Y-%m-%dT%H:%M:%SZ", time.gmtime(epoch)).strip()
Expand Down Expand Up @@ -182,6 +181,14 @@ def trackLog(self, trackResult):
peak = trackResult.peak
lines.append(" Peak level: %.6f" % peak)

# Pre-emphasis status
# Only implemented in whipper (t.pre_emphasis)
if t.pre_emphasis:
preEmph = "Yes"
else:
preEmph = "No"
lines.append(" Pre-emphasis: %s" % preEmph)

# Extraction speed
if trackResult.copyspeed:
lines.append(" Extraction speed: %.1f X" % (
Expand Down

0 comments on commit fb271f0

Please sign in to comment.