-
Notifications
You must be signed in to change notification settings - Fork 31
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
NoTag: reader does not contain an id3 tag #42
Comments
That error comes from |
Your files have ID3v1 tags, and |
Was or Is there a reason why somebody would want to read only |
I mean it does make sense for writing.. yeah so I guess reading for mp3 can use v1v2 by default ? |
Thank you @Serial-ATA, let me try that :) |
Also since Id3 doesn't support writing Id3v1 we either need to label it read-only ? |
Well, I think it just upgrades it to an ID3v2 tag? If that's the case it should be documented, since that could be pretty surprising behavior when you write. |
Is there possibility to lose any information in the conversion from v1 to v2 tags e.g. were any of the fields dropped and are there any incompatibility concerns otherwise e.g. where a field loses data going to v2 from v1 ? If there are any data loss / corruption issues this sounds dicy without making this conversion explicit where the intent is clear from API perspective that it will be lossy (if any) upon write to v2 from v1 read ? Though given there is abstraction at v1v2 at id3 already this is just an upgrade right without any loss / corruption ? |
v1 to v2 is completely lossless since v1 is so basic (only supports title, artist, album, year, comment, track number). The only issue it would theoretically cause is someone is using an incredibly old app that doesn't understand ID3v2 tags, otherwise it's just an overall upgrade since it allows for much more future metadata expansion. |
I am getting the
NoTag: reader does not contain an id3 tag
trying to get the ID3 of a my library. Please tell me if it's an issue forid3
or foraudiotags
:)Node.js'
music-metadata
can parse those with no issues.I have put a few of those files in this drive folder in case that can help.
The text was updated successfully, but these errors were encountered: