gonic icon indicating copy to clipboard operation
gonic copied to clipboard

Is there support for album art embedded in the mp3 files?

Open dabeeeenster opened this issue 5 years ago • 28 comments

Cant seem to find any?

dabeeeenster avatar Nov 23 '19 22:11 dabeeeenster

sorry, not currently supported. I think I would need to find out if the tagging library I'm using supports that https://github.com/nicksellen/audiotags

otherwise a second library will need to be used. I'll have a think about it

sentriz avatar Nov 24 '19 01:11 sentriz

The server works perfect, but it is true that not previewing the covers, it gives the feeling that something is wrong

uGeek avatar Apr 02 '20 13:04 uGeek

this is still something I'm considering. in the meantime, if it's something you're interested in musicbrainz picard has a setting to extract embedded art to a separate file beside your files, which gonic will pickup (personally I prefer this, it saves disk space)

sentriz avatar Apr 02 '20 13:04 sentriz

but ... would clients preview the covers? As you say, it would be a good solution to not make the server heavier

uGeek avatar Apr 02 '20 14:04 uGeek

yeah clients will prefix the covers. if in each of your music folders (albums) you have a "folder.jpg" (or png) gonic will pick it up and show it to the clients

sentriz avatar Apr 02 '20 15:04 sentriz

is very good!!

Totally in agreement with you, I prefer as it is now and that it is much lighter.

In the documentation it should be more visible, it is a very important detail.

Many thanks

uGeek avatar Apr 02 '20 16:04 uGeek

good to hear it's working for you :) you're right about the documentation - thanks!

sentriz avatar Apr 02 '20 17:04 sentriz

I have been trying to export the covers in each directory for two days and I have a lot of problems.

I thought that since the lastfm api is used, when folders are scanned it might gonic to export the cover as folder.jpg in each directory, something like https://github.com/desbma/sacad

That would solve the power to view all the covers.

uGeek avatar Apr 04 '20 11:04 uGeek

It won't download covers for you, only pickup what is already there. But that tool looks perfect. sacad_r your_music 600 folder.jpg and then start a gonic scan. Have you tried this?

sentriz avatar Apr 04 '20 14:04 sentriz

Yes. I have done it and 95% of the covers have downloaded them.

It was a suggestion to perfect the server.

For my gonic it is perfect. Until now I was using Supysonic, but Gonic is much faster. I like much more gonic.

If it had the download of covers integrated, for simplicity and being able to mark the favorite songs, it is without a doubt the best server with api from subsonic.

I have assembled the docker and I am going to do an article and podcast on my blog. Very happy with its operation.

Thank you so much for your work.

uGeek avatar Apr 04 '20 15:04 uGeek

wow thank you very much :) very excited about an article/podcast! ability to favorite songs will be coming soon

sentriz avatar Apr 04 '20 16:04 sentriz

I already published the article and podcast, a complete success. My community has been very interested in this server.

Create the docker to arm and to use it in Raspberry, etc ... And it has more than 10k. https://hub.docker.com/r/ugeek/gonic

  • https://ugeek.github.io//post/2020-05-07-el-mejor-y-m%C3%A1s-rapido-servidor-de-m%C3%BAsica-con-la-api-de-subsonic.html
  • https://ugeek.github.io/blog/post/2020-05-06-gonic-el-mejor-y-mas-ligero-servidor-de-musica-con-api-subsonic.html

Thank you very much for your time and work.

uGeek avatar May 17 '20 20:05 uGeek

wow that's amazing! thank you for the support. i'll have to get a friend who speaks Spanish to listen to it for me :)

sentriz avatar May 17 '20 21:05 sentriz

I'd love to write a pull request for this, but I'm not sure if I have the time. I can at least alert you about tag, which can extract both tags and embedded pictures from relevent audio formats.

danielalvsaaker avatar Aug 19 '20 16:08 danielalvsaaker

I'd love to write a pull request for this, but I'm not sure if I have the time. I can at least alert you about tag, which can extract both tags and embedded pictures from relevent audio formats.

i would have loved to have used tag because it's pure go, currently gonic uses nicksellen/audiotags which is a go wrapper for taglib, a c++ project. so tag could be faster without the interop overhead. but unfortunately tag doesn't support extracting song durations which is important for some subsonic clients.

maybe it could be possible to run both at scan time, but it may slow things down a good bit

sentriz avatar Aug 21 '20 16:08 sentriz

I have just installed gonic for the first time. I like its simplicity and speed, but I really miss the ability to display album art from mp3 tags.

Perhaps tag could be expanded to read song duration?

pacija avatar Sep 10 '20 10:09 pacija

Same for me, also miss this feature.

helmut72 avatar Dec 31 '20 09:12 helmut72

Really need this feature, I have 2TB of anime songs, they have a scans folder for images, but copy each img out won't be easy for a large collection, also have to rename them too

Anyway to imply a custom path for scans folder and select first image as album art?

image

AlexanderHel avatar Oct 31 '22 03:10 AlexanderHel

@AlexanderHel

as a workaround, if you have access to a unix like environment, you could automatically find all Scan folders with for example a folder.png in them. and create a symbolic link to the parent of those folders so that gonic could pick them up

for example

find /path/to/your/music/ -type d -name "Scans" -exec ln -s -t {}/.. Scans/folder.png \;

sentriz avatar Nov 02 '22 20:11 sentriz

thank you, sadly the name for the image is different not all of them is the same and obvious not folder.png

I work around with a python script to find scans folder, copied the first image to previous folder and rename it to folder.png, but the script failed when I have too many folder path, so I gave up and wait for future embedded album art update lol

Also because I noticed not all CD have a scans folder so embedded cover still better

AlexanderHel avatar Nov 03 '22 08:11 AlexanderHel

This feature and external auth (https://github.com/sentriz/gonic/issues/44#issuecomment-1364588428) would complete Gonic for me.

helmut72 avatar Jan 06 '23 11:01 helmut72

@sentriz I've fork audiotags and add support for read embedded images for some media formats.

PSyton avatar Jan 08 '23 09:01 PSyton

I am utilizing MusicBrainz Picard to process all my audio tracks before their are moved into to Gonic's . However, I am requesting a feature that permits extraction of cover images and additional associated data directly from the file's meta tags and then use folder image as fallback.

For reference, the Taglib library presents itself as a potential solution in this context. I don't known Go-lang, if i did i would contribute.

mattiasghodsian avatar Aug 31 '23 18:08 mattiasghodsian

Instead of directly reading album art from ID3 tags, would it maybe be an option for the importer / scanner to detect album art inside songs and extract them to the cover.jpg file inside the album folder? Obviously this would need to be behind a toggle, but it might require less changes than reading and passing on to clients, and additionally be more in like with how gonic is designed to function.

bytebone avatar Jan 25 '24 17:01 bytebone

Instead of directly reading album art from ID3 tags, would it maybe be an option for the importer / scanner to detect album art inside songs and extract them to the cover.jpg file inside the album folder? Obviously this would need to be behind a toggle, but it might require less changes than reading and passing on to clients, and additionally be more in like with how gonic is designed to function.

i think this could be a problem for non-album collections, i don't have any albums only genre/random name folders with gonic. I think extracting the images from the audio files and having them in the cache folder with a uuid hash matching the track is a better alternative

mattiasghodsian avatar Jan 25 '24 17:01 mattiasghodsian

While TagLib itself has a way of retrieving the embedded cover art buffer, the Go abstraction didn't implement it, so using it would be using the underlying C bindings, and at that point it's wise to look at other approaches.

I've looked at how Navidrome does it, and it seems they built their own TagLib wrapper to get that functionality.

NyaomiDEV avatar Mar 18 '24 21:03 NyaomiDEV

the fork of audiotags which gonic uses also supports embedded art fwiw

sentriz avatar Mar 19 '24 00:03 sentriz

Just commenting here asking if there is going to be anything in place that solves this issue. As of now all my songs have tagged album art covers with a "Scans" folder containing the images. Will this continue to be the case for gonic?

Album Release (catalog)/
├── Disc 1
│   ├── 01 song.flac
├── Disc 2
│   ├── 01 song.flac
├── Disc 3
│   ├── 01 song.flac
└── Scans
    └── Box Front.jpg

meowmeowofthemeows avatar Apr 07 '24 14:04 meowmeowofthemeows