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

Auto Ripping - doesn't start, not always available #46

Closed
DonnKey opened this issue Aug 14, 2019 · 8 comments
Closed

Auto Ripping - doesn't start, not always available #46

DonnKey opened this issue Aug 14, 2019 · 8 comments
Labels

Comments

@DonnKey
Copy link

DonnKey commented Aug 14, 2019

Describe the bug
Two (possibly) related issues:

  1. Auto start simply doesn't auto start.
  2. In a new configuration, Auto Start is greyed out until app is restarted.

To Reproduce
Steps to reproduce the behavior:
Version 20190423.

  1. Try to get it to auto start ripping. (This works fine on 1.0.32 on the same HW.) It won't.
  2. Options->General Settings -> Create New Configuration.
    Then Ripper->Settings->Automation "Read CD Contents on Insert" and "Start Ripping Automatically" are greyed out.
    Restart application and they are available.

Expected behavior
Options should be available just after creating a new configuration (no restart should be needed).
Auto Start Ripping should actually do that.

Screenshots
If applicable, add screenshots to help explain your problem.

System (please complete the following information)

  • OS: Windows 10 1803 (17134.885)
  • CPU: Intel Core i3-8100 3.60 GHz.
  • RAM: 8Gb

Additional context
Add any other context about the problem here.

@DonnKey DonnKey added the bug label Aug 14, 2019
@enzo1982
Copy link
Owner

Thank you for reporting this!

The second issue (automatization options not being available after creating a new configuration) is fixed now with the 7e33167 commit.

As for the first issue, there have been some issues with this feature on Windows in the past (not only with the 1.1 development versions, but also with 1.0.x). In some cases, Windows just does not seem to be sending the media arrival message to all applications. Though it's strange that it works for you with fre:ac 1.0.32.

If you have been using the fre:ac 1.1 64 bit version, could you try if the 32 bit version behaves better?

Also, is there anything remarkable about your CD drives' setup? Multiple drives or something?

@DonnKey
Copy link
Author

DonnKey commented Aug 17, 2019 via email

@DonnKey
Copy link
Author

DonnKey commented Dec 17, 2019

I just tried the very recent Beta. With the correct setup it works fine for me except for the original bug mentioned here: it doesn't autostart the rip. It does get the new-disk notification (it populates the joblist without help), but it doesn't actually start the rip until "Start encoding" is clicked. I didn't repeat the tests, but last time I discovered that this was because there was no metadata whatsoever available (nothing on the CD, and web search disabled because I know there's nothing on the web either). It did, then, autostart if the CD had metadata.

Once it starts, it's great, and quite fast enough. I have an adequate solution to the disk number problem (the time/date is enough if I read the disks in order).

@enzo1982
Copy link
Owner

Thank you for trying the beta and also for your longer comment from August! Unfortunately, I didn't have time to write a detailed answer yet as I was quite busy preparing the beta.

You are right. Ripping does not start automatically when there is no metadata available. This actually was a feature request by another user and implemented earlier this year. The reasoning is that without metadata, rips from different discs would land in the same folder and/or overwrite each other (as you already experienced as well).

Though this cannot happen when the <currentdate> and <currenttime> placeholders are used in the filename pattern. So I guess I will add an exception for when this is the case.

Regarding your comments from your August post (again, sorry for not answering this earlier):

Reading the ISRC for each track can cause some delay and/or audible seeking when adding the tracks to the joblist. The data that possibly contains the ISRC values is read in any case, even if the disc does not have any ISRC values. I think audio books generally will not have ISRCs, as these are issued only for music tracks as far as I know. So you might just want to disable reading ISRCs if you are dealing mostly with audio books.

I can confirm the bug where the ripping process is not stopped when ejecting the disc and fre:ac cannot be quit then. I'll try to fix this for the next beta. Thank you for pointing this out!

Likewise for pressing the eject button while adding disc contents to the joblist.

Regarding a sequential disc number counter, that's already on my list of possible future features. So it's likely this will come with fre:ac 1.1.x or maybe 1.2. This will also include options to re-number tracks from an arbitrary starting point or use a disc-spanning track counter.

@DonnKey
Copy link
Author

DonnKey commented Dec 21, 2019

Thanks! I'll look forward to all the changes.

With respect to auto-start in the absence of metadata: for most (non-audiobook) users, auto starting the rip in the absence of metadata is probably the right thing (particularly if they didn't know in advance that the CD was odd!). But then there's the audiobook crowd (and some home-brew CDs, probably). I'd personally be just as happy with another checkbox of the form "Allow autostart without metadata", rather than trying to infer the user's intent from the path name. That alternative also announces your intent clearly, so it won't be perceived of as a bug or obscure behavior - the user would perceive it as a "a-ha" sort of feature, and it would also implicitly guide the user to think about why that's an issue and what would work best for them.

Some sort of message of the form "autostart not done because metadata missing" would also be good guidance and probably clear up a lot of user mysteries. (Some sort of implied message, such as red in the joblist, or making the message I suggested above turn red if it applies, would be fine too.) Whatever works in the balance between needs and your available time is fine.

@enzo1982
Copy link
Owner

Beta 2 is now available and should auto start ripping even when no metadata is available, as long as <currenttime> is used in the output pattern.

It also should improve behavior when ejecting a disc while ripping or while adding tracks to the joblist.

@DonnKey
Copy link
Author

DonnKey commented Jan 20, 2020

(Finally got to testing.) Works as I had hoped. Thanks very much!

@enzo1982
Copy link
Owner

Great! Thank you for reporting back!

Closing this issue then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants