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

Various improvements #18

Merged
merged 5 commits into from
Feb 5, 2023
Merged

Various improvements #18

merged 5 commits into from
Feb 5, 2023

Conversation

MikeLund
Copy link
Contributor

@MikeLund MikeLund commented Feb 1, 2023

@MikeLund MikeLund changed the title Don't use localmedia; query using full path; create organized collect… Various improvements Feb 2, 2023
@MikeLund MikeLund marked this pull request as draft February 4, 2023 16:59
@MikeLund
Copy link
Contributor Author

MikeLund commented Feb 4, 2023

I'm not sure clear_posters(metadata) and clear_art(metadata) is good to have as it makes identifying tasks take longer (it might increase IO?), whereas normal scheduled task to remove unused bundles should instead fix that situation... anyone have input regarding that?

Seems to give heavier impact to IO and shouldn't be needed in most cases I assume
@MikeLund MikeLund marked this pull request as ready for review February 4, 2023 19:10
Don't need to import Helpers when not using its clear_ methods.
@Darklyter
Copy link
Owner

Hey there, I didn't realize this was here until I saw you post about it. Been away from my system for a bit. I'll merge it when I get back in town. And thanks!

@Darklyter Darklyter merged commit a8527b1 into Darklyter:main Feb 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants