beets-alternatives icon indicating copy to clipboard operation
beets-alternatives copied to clipboard

ebmedart doesn't seem to work

Open JOJ0 opened this issue 1 year ago • 5 comments

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 avatar Apr 16 '23 12:04 JOJ0

@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?

JOJ0 avatar Apr 26 '23 09:04 JOJ0

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

https://github.com/geigerzaehler/beets-alternatives/blob/288299e3aa9a1602717b04c28696fce5ce4259bf/beetsplug/alternatives.py#L320-L330

wisp3rwind avatar Apr 26 '23 21:04 wisp3rwind

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.

geigerzaehler avatar Apr 27 '23 06:04 geigerzaehler

@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!

wisp3rwind avatar Apr 27 '23 15:04 wisp3rwind

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!

JOJ0 avatar Apr 28 '23 11:04 JOJ0