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

redacted/waffles ready #288

Open
4 of 7 tasks
MerlijnWajer opened this issue Aug 6, 2018 · 13 comments
Open
4 of 7 tasks

redacted/waffles ready #288

MerlijnWajer opened this issue Aug 6, 2018 · 13 comments
Assignees
Labels
Accepted Accepted issue on our roadmap Story Issue used to track progress on multiple related issues in a single place
Milestone

Comments

@MerlijnWajer
Copy link
Collaborator

MerlijnWajer commented Aug 6, 2018

I am trying to make a list of tickets we should finish before we can work with redacted/waffles on the logger part and then have whipper approved by both sites.

Anything else? We might have to rework the logger interface so that it can also call (external) binaries.

I didn't make a Milestone for this because (AFAIK) tickets can be only part of one milestone.

@MerlijnWajer MerlijnWajer self-assigned this Aug 6, 2018
@MerlijnWajer
Copy link
Collaborator Author

@JoeLametta - any comments? I'm trying to get the ball rolling again on this.

@peaveyman
Copy link

We need to find out for sure which logger is going to be acceptable for redacted. And if redacted is good with it waffles will be.

@MerlijnWajer
Copy link
Collaborator Author

It seems like the current logger is fine - so we don't need #117. It's unclear if we will need #135. #221 would be nice to have, but for testing purposes we don't need it. #294 is important, though. #174 seems not that important for red right now since it mostly affects CDs with extra data, but we should get that fixed soon as well.

@peaveyman
Copy link

I can't see how 135 and 221 would be needed.

@JoeLametta
Copy link
Collaborator

  1. Loggers #117 I don't consider it a blocker: I'll surely update the README to clarify logger plugins current status. I may also update the morituri-eaclogger (or discontinue it).
  2. Add a requirements.txt file #221 I would say it's 99% done: again, not a blocker.
  3. Ensure that multi-session CDs are still read correctly by whipper #174 Would be nice having this fixed
  4. Allow plugins from system directories #135 Would be nice having this fixed: not a blocker.
  5. Go over cdparanoia error types and see if there is one that indicates that rips were definitely lossy #294 I think this is needed.

Clear tutorials (more up to the websites perhaps)

Maybe #73?

@JoeLametta JoeLametta added this to the 1.0 milestone Nov 3, 2018
@JoeLametta JoeLametta added the Story Issue used to track progress on multiple related issues in a single place label Nov 12, 2018
@itismadness
Copy link
Contributor

itismadness commented Nov 30, 2018

So in looking to implement this within a logchecker for Orpheus, one of the biggest hurdles I think we face is just knowing what are the possible logs that whipper may output that a user may check against. This includes a perfect log, but also what a log of a rip that has totally failed (or failed as much as whipper would allow while still producing a log). I can get an idea of it looking through the logger.py module that produces the YAML, but just concrete examples would be wonderfully helpful.

(Thanks for making the default log produced using YAML and not just straight text which has made parsing these things way easier than EAC/XLD).

@JoeLametta JoeLametta added the Accepted Accepted issue on our roadmap label Feb 13, 2019
@MerlijnWajer
Copy link
Collaborator Author

Alright. Going to try to get this rolling ASAP now. Remaining three tickets are all assigned to me it seems. ;)

@nafanz
Copy link

nafanz commented Jul 19, 2019

It is wonderful that it was approved at Orpheus.
I hope that it will also be available for RED and Waffles.

@peaveyman
Copy link

I can't speak for RED but as for Waffles, when it gets back online you can expect Whipper to be an approved ripper.

@JoeLametta
Copy link
Collaborator

Even if it's an upstream bug I think #234 needs to be fixed too because it affects a big number of users.

@7596ff
Copy link

7596ff commented Jun 5, 2020

Is this still a priority?

@JoeLametta
Copy link
Collaborator

JoeLametta commented Jun 6, 2020

Is this still a priority?

Hi, those are just some of the issues scheduled for milestone 1.0. Unfortunately I don't have enough time / experience to handle everything on my own (that's why progress is slow) but I try to review every issue / pull request in a timely manner.
Of course every contribution is encouraged and appreciated.

@lighterowl
Copy link

I can't speak for RED but as for Waffles, when it gets back online you can expect Whipper to be an approved ripper.

It never did. 😢

@in-plaintext
Copy link

What is the current state of the program on RED?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted Accepted issue on our roadmap Story Issue used to track progress on multiple related issues in a single place
Projects
None yet
Development

No branches or pull requests

8 participants