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

Ripping series only outputting one file #163

Closed
2 tasks done
GamerBene19 opened this issue Apr 30, 2022 · 12 comments
Closed
2 tasks done

Ripping series only outputting one file #163

GamerBene19 opened this issue Apr 30, 2022 · 12 comments
Assignees
Labels
bug Something isn't working Fixed

Comments

@GamerBene19
Copy link

  • I have checked the wiki
  • I have searched for similar issues

Describe the bug
When ripping a series only one file is put into the output directory

Expected result
I would expect all episodes of a series to be present with name something like Sample Series (1972) - Episode 1.mkv

To Reproduce

  1. Set method to mkv (I have it set to that, but don't think its relevant here)
  2. Rip disc containing multiple episodes of a series
  3. Observe all episodes beeing put into raw but only one (I think the first) beeing put into completed

Environment

Ubuntu 20.04.4, on commit 8d32bab757f090d474d85c579037021771d19fec of branch v2_devel

Log file (cut to relevant section at the end)

Sample Series_DISC1.log

@GamerBene19 GamerBene19 added the bug Something isn't working label Apr 30, 2022
@1337-server 1337-server reopened this Apr 30, 2022
@1337-server
Copy link
Owner

Reopened as it needs testing fully, but this should fix the issue.

@1337-server 1337-server added the Awaiting feedback Waiting for user to test fixes label Apr 30, 2022
@GamerBene19
Copy link
Author

as it needs testing fully

How would I do that? Do just I update my existing install? If so: How?

@1337-server
Copy link
Owner

Yes its just a small update to your install.
This tells you how to update.
https://github.com/automatic-ripping-machine/automatic-ripping-machine/wiki/upgrading

@GamerBene19
Copy link
Author

GamerBene19 commented Apr 30, 2022

I've tried out the changes. They don't seem to work. I don't get any files output anymore (could also be because of the other changes made since install though).

Here's the relevant log section again
Sample Series.log

As far as I can tell it tries to move the files from my transcode directory. But there are no files there as I skip transcoding. Note that the log also does not say anything about skipping transcoding anymore.

Edit: It seems like ripping movies fails for the same reason.

@1337-server
Copy link
Owner

Ok, I think I got it. I'm going to test a bit more before I push, but I've narrowed it down to this line.

move_files_post(hb_out_path, job)

It pushes the out_path into move files, but if transcoding is skipped, in_path should be used. As out_path (TRANSCODE_PATH) will be empty.

I'll update when I have fully tested a fix.

@GamerBene19
Copy link
Author

My initial issue is solved (all episodes of a series are copied to the output directory). However when a movie now has more than one file. The largest one is beeing left in the extras dir as title_t<number> instead of being copied (and renamded to <movie title> (<year>).mkv in the folder above extras.

@1337-server 1337-server reopened this May 1, 2022
1337-server added a commit that referenced this issue May 1, 2022
fixing code smells
@1337-server
Copy link
Owner

This should finally fix it, let me know if not.

@GamerBene19
Copy link
Author

Thanks for the quick fix. Sadly I won't be able to test this till at least friday. I'll give you an update then!

@1337-server
Copy link
Owner

No worries, there is no rush.

@GamerBene19
Copy link
Author

GamerBene19 commented May 5, 2022

I just tested a movie (with the current version). It seems to work. I will test series shortly but I expect them to work too

However I've noticed something different. Before I could change the title during ripping and the output would get placed into the folder with the new title (the old folder that was created at the beginning still existed, but you could just delete it afterwards). Now, if I don't update the title before the ripping begins the file will not be placed into a folder with the new name.

1337-server added a commit that referenced this issue May 6, 2022
@1337-server
Copy link
Owner

It should now update if you set a custom title, or update title from search

1337-server added a commit that referenced this issue May 6, 2022
@GamerBene19
Copy link
Author

So after a bit of testing this weekend I think this issue can be closed. Series are ripped as expected and other behaviour is the same (again)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Fixed
Projects
None yet
Development

No branches or pull requests

2 participants