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

Reading returns empty list (both ID3v2.3 and ID3v2.4) #210

Closed
lazka opened this issue Jan 31, 2015 · 8 comments
Closed

Reading returns empty list (both ID3v2.3 and ID3v2.4) #210

lazka opened this issue Jan 31, 2015 · 8 comments
Labels
bug

Comments

@lazka
Copy link
Member

@lazka lazka commented Jan 31, 2015

Originally reported by: Anonymous


I attached links to three files. I cannot read 'La Mulateada' or 'Si Tu Quisieras', they have ID3v2.4 respective ID3v2.3 according to MP3diags. I can read the third file with Mutagen. It has ID3v2.4.

https://dl.dropboxusercontent.com/u/13912268/Beam/Share/02%20-%20La%20Mulateada.mp3
https://dl.dropboxusercontent.com/u/13912268/Beam/Share/03%20-%20Si%20Tu%20Quisieras.mp3
https://dl.dropboxusercontent.com/u/13912268/Beam/Share/04%20-%20Los%20Munequitos.mp3


Info from MP3Diags:

La Mulateada:

ID3V2.3.0
padding=1495, unsynch=yes; frames: TSSE="LAME 64bits version 3.99.3 (http://lame.sf.net)", TRCK="2", TPE1="Carlos Di Sarli", TALB="Tango Argentino - Sus Primeros Exitos - Vol 3", TIT2="La Mulateada", TCON="Milonga", TLEN="141200", TXXX="replaygain_reference_loudness 89.0 dB", RVA2 size=12, TXXX="replaygain_track_gain +0.475000 dB", TXXX="replaygain_track_peak 0.410779", TXXX="MP3GAIN_MINMAX 000,205", TXXX="MP3GAIN_UNDO +005,+005,N", POPM size=9, TPE2="Carlos Di Sarli", COMM size=25, TBPM="108", TDRC="1941"

ID3V1
ID3V1.1b, La Mulateada, Carlos Di Sarli, Tango Argentino - Sus Primeros

Si Tu Quisieras:

ID3V2.4.0
padding=1509, unsynch=yes; frames: TSSE="LAME 64bits version 3.99.3 (http://lame.sf.net)", TRCK="3", TPE1="Carlos Di Sarli", TALB="Tango Argentino - Sus Primeros Exitos - Vol 3", TIT2="Si T˙ Quisieras", TCON="113", TLEN="159106", TXXX="replaygain_reference_loudness 89.0 dB", RVA2 size=12, TXXX="replaygain_track_gain -0.620000 dB", TXXX="replaygain_track_peak 0.489175", TXXX="MP3GAIN_MINMAX 090,206", TXXX="MP3GAIN_UNDO +004,+004,N", POPM size=9, TPE2="Carlos Di Sarli", COMM size=25, TDRC="1943"

ID3V1
ID3V1.1b, Si T˙ Quisieras, Carlos Di Sarli, Tango Argentino - Sus Primeros, Tango

Los Munequitos:

ID3V2.4.0
padding=1510, unsynch=yes; frames: TSSE="LAME 64bits version 3.99.3 (http://lame.sf.net)", TRCK="4", TPE1="Carlos Di Sarli", TALB="Tango Argentino - Sus Primeros Exitos - Vol 3", TIT2="Los MuÒequitos", TCON="113", TLEN="195800", TXXX="replaygain_reference_loudness 89.0 dB", RVA2 size=12, TXXX="replaygain_track_gain +0.565000 dB", TXXX="replaygain_track_peak 0.411062", TXXX="MP3GAIN_MINMAX 069,205", TXXX="MP3GAIN_UNDO +005,+005,N", POPM size=9, TPE2="Carlos Di Sarli", COMM size=25, TDRC="1943"

ID3V1
ID3V1.1b, Los MuÒequitos, Carlos Di Sarli, Tango Argentino - Sus Primeros, Tango


SCRIPT:

import mutagen

file1 = "02 - La Mulateada.mp3"
file2 = "03 - Si Tu Quisieras.mp3"
file3 = "04 - Los Munequitos.mp3"

audio1 = mutagen.File(file1, easy=True)
print audio1
print

audio2 = mutagen.File(file2, easy=True)
print audio2
print

audio3 = mutagen.File(file3, easy=True)
print audio3
print


Result:
python MutagenTest2.py
{}

{}

{'album': [u'Tango Argentino - Sus Primeros Exitos - Vol 3'], 'performer': [u'Carlos Di Sarli'], 'artist': [u'Carlos Di Sarli'], 'title': [u'Los Mu\xf1equitos'], u'replaygain_track_peak': [u'1.644165'], 'genre': [u'Tango'], 'length': [u'195800'], u'replaygain_track_gain': [u'+0.564453 dB'], 'date': [u'1943'], 'tracknumber': [u'4']}


@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Feb 1, 2015

Original comment by Christoph Reiter (Bitbucket: lazka, GitHub: lazka):


thanks

@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Feb 1, 2015

Original comment by Christoph Reiter (Bitbucket: lazka, GitHub: lazka):


The "POPM:Banshee" frame in the first two files is marked unsychronized but isn't.

@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Feb 2, 2015

Original comment by Mikael Holber (Bitbucket: holber, GitHub: Unknown):


Is this related only to my setup/specific version of Banshee that I tagged my music with?

@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Feb 2, 2015

Original comment by Mikael Holber (Bitbucket: holber, GitHub: Unknown):


I accidently posted this issue as anonymous.

@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Feb 2, 2015

Original comment by Christoph Reiter (Bitbucket: lazka, GitHub: lazka):


Probably, but mutagen could be less strict in this case.

@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Feb 2, 2015

Original comment by Mikael Holber (Bitbucket: holber, GitHub: Unknown):


Okay, let me know if I can help with anything.

@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Feb 18, 2015

Original comment by Mikael Holber (Bitbucket: holber, GitHub: Unknown):


Hi! Is there any update on this issue? Do you have any plan for implementation?

@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Feb 18, 2015

Original comment by Christoph Reiter (Bitbucket: lazka, GitHub: lazka):


id3: only warn in case a synch safe marked frame isn't sync safe. (Fixes issue #210)

@lazka lazka added major bug labels Apr 7, 2016
@lazka lazka closed this Apr 7, 2016
lazka added a commit that referenced this issue Apr 7, 2016
…xes isse #210)

There is a good chance the data is fine in this case and the writter just forgot to unsynch the data.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.