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

ebmedart doesn't seem to work #69

Open
JOJ0 opened this issue Apr 16, 2023 · 5 comments
Open

ebmedart doesn't seem to work #69

JOJ0 opened this issue Apr 16, 2023 · 5 comments

Comments

@JOJ0
Copy link

JOJ0 commented Apr 16, 2023

What do I need to configure for convert and alternatives plugin to make my FLAC files in the alternative version embeded with art? As far as I understood the docs, it should be default. I've set embedart: yes in my convert config stanza but still my encoded alternative version mp3 files don't have coverart embeded.

  • My art is embeded in the original files
  • I don't have cover.ext image files lying around in my album folders. Could this be the problem?

Thanks and sorry for being brief. If you think it makes sense, I'll certainly deliver detailed logs, commands and so on.

@JOJ0
Copy link
Author

JOJ0 commented Apr 26, 2023

@geigerzaehler just a quick one: Before I have to start reading this plugins' code: See my bulletpoints. Is that the reason embedding art does not work?

@wisp3rwind
Copy link
Collaborator

Yes, I'm pretty sure that this is the problem:

def sync_art(self, item, path):
""" Embed artwork in the destination file.
"""
album = item.get_album()
if album:
if album.artpath and os.path.isfile(syspath(album.artpath)):
self._log.debug("Embedding art from {} into {}".format(
displayable_path(album.artpath),
displayable_path(path)))
art.embed_item(self._log, item, album.artpath,
itempath=path)

@geigerzaehler
Copy link
Owner

As @wisp3rwind has pointed out this plugin only actively embeds album art if the art is stored as a file on disk. Otherwise, as in your case, the plugin relies on the convert plugin to extract the art and embed it in the transcoded target file. You’ve seemed to have configured convert correctly. This means that either this plugin is not using convert correctly or there is an issue with convert itself. (Maybe also a combination of both.)

@JOJ0: I’m not actively committing fixes to this project anymore. Maybe @wisp3rwind is interested in providing a fix. But I would also like to encourage you to dig into the code yourself and see if you can’t find the problem and provide a fix. I’m happy to support your contribution, answer questions and provide reviews.

@wisp3rwind
Copy link
Collaborator

@JOJ0: I’m not actively committing fixes to this project anymore. Maybe @wisp3rwind is interested in providing a fix. But I would also like to encourage you to dig into the code yourself and see if you can’t find the problem and provide a fix. I’m happy to support your contribution, answer questions and provide reviews.

Same here: I don't think I'll be able to look into this myself anytime soon (it's also not a problem that I personally encounter in my usage of beets). Happy to review a PR, though!

@JOJ0
Copy link
Author

JOJ0 commented Apr 28, 2023

Hi y'all, I'm actually enjoying a kind of beets coding break as well haha. Thanks for your replies and pointers. Also I found a way to suit my current needs without this plugin. Still I think it would be an essential feature. I might pick it up in the future if need be.

Thanks for maintaining a great plugin to you both!

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

No branches or pull requests

3 participants