IPB

Welcome Guest ( Log In | Register )

Flac Doesn't Let Me Embed This Cuesheet, What's wrong with it?
smz
post Feb 6 2005, 13:43
Post #1





Group: Members
Posts: 601
Joined: 15-February 04
From: Venezia, Italia
Member No.: 12025



Flac (1.1.2) doesn't let me embed the follwing cuesheet

CODE
PERFORMER "Performer Name"
TITLE "Album Title"
FILE "mytestfile.wav" WAVE
 TRACK 01 AUDIO
   TITLE "Track01 Title"
   PERFORMER "Performer Name"
   INDEX 01 00:00:00

   ...

with this error:

CODE
ERROR initializing encoder
state= FLAC_STREAM_ENCODER_INVALID_METADATA


Is this a bug or there is something wrong with my cuesheet that I can't spot?

Cheers

Sergio


Edit: spelling. If also a mod could fix the topic title... thanks! (Done, thanks to the mod!)

Edit2: removed great part of the cuesheet as the problem was not with it (see subsequent posts), to make the thread more readable.

Edit3: Modified cuesheet example so that RIAA doesn't sue me, in case I die...

This post has been edited by smz: Feb 9 2005, 02:34


--------------------
Sergio
Revox B150 + (JBL 4301B | Sennheiser HD430)
Go to the top of the page
+Quote Post
 
Start new topic
Replies
jcoalson
post Feb 8 2005, 19:04
Post #2


FLAC Developer


Group: Developer
Posts: 1526
Joined: 27-February 02
Member No.: 1408



I see the problem, but I'm not sure what to do about it. here's the problem:

1. the wave file has a number of samples that is not evenly divisible by 588, i.e. not a full CD sector

2. when importing a cuesheet, flac creates a 'leadout' track for the cuesheet implicitly based on the size of the wave file, since cuesheets do not explicitly describe the leadout track.

3. when flac is told to apply a cuesheet to a file, if the input is 2 channels, 16 bps, and 44.1kHz it assumes that the cuesheet is applying to CD-DA and applies more strict checks to it.

4. flac sees a leadout track that is not on a sector boundary and complains that it is invalid in a vague way.

5. metaflac --import-cuesheet-from works because it is not as strict at checking against the CD-DA spec (it actually should be as strict, that's a bug), namely it does not check that the leadout is on a sector boundary.

maybe flac/metaflac need an option to tell it to not check the CD-DA subset on a cuesheet.

Josh
Go to the top of the page
+Quote Post
smz
post Feb 9 2005, 02:26
Post #3





Group: Members
Posts: 601
Joined: 15-February 04
From: Venezia, Italia
Member No.: 12025



Josh, thank-you for analyzing the problem.

Here are my considerations about it:

QUOTE (jcoalson @ Feb 8 2005, 07:04 PM)
1. the wave file has a number of samples that is not evenly divisible by 588, i.e. not a full CD sector

Being evenly divisible by 588 is not required by the WAV standard, AFAIK.

QUOTE (jcoalson @ Feb 8 2005, 07:04 PM)
2. when importing a cuesheet, flac creates a 'leadout' track for the cuesheet implicitly based on the size of the wave file, since cuesheets do not explicitly describe the leadout track.

I can't understand that. If the cuesheets "standard" doesn't calls for an explicitly defined leadout, why should FLAC do? I've always thought FLAC as a lossless codec, bit perfect, independently of the content and/or size of my input file. I've thought of the embedded cuesheet just as metatdata to be parsed by an external application, associated to specific seekpoints. And shouldn't be the businness of a CD burning (or emulation) application to eventually deal with a possibly incomplete last sector of the PCM stream?

QUOTE (jcoalson @ Feb 8 2005, 07:04 PM)
3. when flac is told to apply a cuesheet to a file, if the input is 2 channels, 16 bps, and 44.1kHz it assumes that the cuesheet is applying to CD-DA and applies more strict checks to it.

It shouldn't IMHO and if I'm not missing some crucial point.

QUOTE (jcoalson @ Feb 8 2005, 07:04 PM)
4. flac sees a leadout track that is not on a sector boundary and complains that it is invalid in a vague way.

I think the message should be made more clear... or just shuldn't be necessary at all, always IMO.

QUOTE (jcoalson @ Feb 8 2005, 07:04 PM)
5. metaflac --import-cuesheet-from works because it is not as strict at checking against the CD-DA spec (it actually should be as strict, that's a bug), namely it does not check that the leadout is on a sector boundary.

Not a bug at all, from my point of view, at least if it doesn't have any weired side-effect in some situation I can't now foresee.

QUOTE (jcoalson @ Feb 8 2005, 07:04 PM)
maybe flac/metaflac need an option to tell it to not check the CD-DA subset on a cuesheet.

... and this option should be made the default! biggrin.gif


Please take my consideration for what they are: just the opinion of an otherwise very satisfied user of your wanderful work. But I've really the strong feeling that something is wrong with your assumptions that a FLAC file with an embedded cuesheet should obey to the rules of a CD-DA.

Cheers!

Sergio


--------------------
Sergio
Revox B150 + (JBL 4301B | Sennheiser HD430)
Go to the top of the page
+Quote Post

Posts in this topic
- smz   Flac Doesn't Let Me Embed This Cuesheet   Feb 6 2005, 13:43
- - BruRob   Your third line reads: FILE "Nearness of You-...   Feb 6 2005, 14:56
|- - smz   QUOTE (BruRob @ Feb 6 2005, 03:56 PM)Your thi...   Feb 6 2005, 15:23
|- - smz   Sorry to bump this post, but I think I have some (...   Feb 6 2005, 20:11
|- - jcoalson   I probably cannot tell anything without having the...   Feb 7 2005, 05:28
|- - smz   QUOTE (jcoalson @ Feb 7 2005, 06:28 AM)I prob...   Feb 7 2005, 12:32
|- - lexor   QUOTE (smz @ Feb 7 2005, 03:32 AM)QUOTE (jcoa...   Feb 7 2005, 13:44
|- - smz   QUOTE (lexor @ Feb 7 2005, 02:44 PM)try and s...   Feb 7 2005, 14:16
|- - smz   Does anybody knows a tool to validate the structur...   Feb 8 2005, 12:04
|- - rjamorim   QUOTE (smz @ Feb 8 2005, 09:04 AM)Does anybod...   Feb 8 2005, 13:21
- - Synthetic Soul   I just tested your WAVE and CUE and got the same r...   Feb 8 2005, 13:21
- - Synthetic Soul   QUOTE (rjamorim @ Feb 8 2005, 12:21 PM)QUOTE ...   Feb 8 2005, 13:33
- - jcoalson   I see the problem, but I'm not sure what to do...   Feb 8 2005, 19:04
|- - smz   Josh, thank-you for analyzing the problem. Here a...   Feb 9 2005, 02:26
- - Synthetic Soul   QUOTE (smz @ Feb 9 2005, 01:26 AM)QUOTE (jcoa...   Feb 9 2005, 11:23
- - kjoonlee   IMHO the assumption(s) that flac and metaflac make...   Feb 9 2005, 15:22
|- - smz   QUOTE (kjoonlee @ Feb 9 2005, 03:22 PM)... Th...   Feb 9 2005, 15:41
|- - rjamorim   QUOTE (kjoonlee @ Feb 9 2005, 12:22 PM)One vo...   Feb 9 2005, 18:04
- - jcoalson   the majority of cuesheet use is via cuesheets ripp...   Feb 9 2005, 18:56
|- - rjamorim   QUOTE (jcoalson @ Feb 9 2005, 03:56 PM)the ma...   Feb 9 2005, 19:01
- - kjoonlee   The cuesheet format screams out "CD-DA"....   Feb 10 2005, 12:51
|- - smz   QUOTE (kjoonlee @ Feb 10 2005, 12:51 PM)The c...   Feb 10 2005, 13:17
|- - rjamorim   QUOTE (smz @ Feb 10 2005, 10:17 AM)Correct. W...   Feb 12 2005, 03:24
|- - smz   QUOTE (rjamorim @ Feb 12 2005, 03:24 AM)QUOTE...   Feb 12 2005, 03:38
|- - rjamorim   QUOTE (smz @ Feb 12 2005, 12:38 AM)I didn...   Feb 12 2005, 17:11
- - m0rbidini   I agree with rjamorim and smz. FLAC is making assu...   Feb 10 2005, 13:18
- - m0rbidini   Er, correction. I wrote: QUOTE Cue sheets don...   Feb 10 2005, 13:25
- - Synthetic Soul   QUOTE (m0rbidini @ Feb 10 2005, 12:25 PM)...   Feb 10 2005, 13:46
- - jcoalson   how is the cuesheet duration specified if there is...   Feb 10 2005, 16:24
- - Synthetic Soul   Point taken. You could ascertain whether the last...   Feb 10 2005, 16:30
- - m0rbidini   Meh, I had it "more or less right" the f...   Feb 11 2005, 15:09
- - Synthetic Soul   Actually, on re-reading, I think you said it just ...   Feb 11 2005, 15:23
|- - smz   I would like to make with you all a sanity check o...   Feb 12 2005, 01:23
|- - smz   One more consideration: If foobar2000 easly accep...   Feb 12 2005, 02:16
|- - rjamorim   ...   Feb 12 2005, 03:28
- - jcoalson   following up, this is changed in CVS, output now l...   Aug 24 2005, 01:27
- - Synthetic Soul   Thanks Josh. As you say, it's up to the user ...   Aug 24 2005, 09:40


Reply to this topicStart new topic
1 User(s) are reading this topic (1 Guests and 0 Anonymous Users)
0 Members:

 



RSS Lo-Fi Version Time is now: 23rd August 2014 - 11:39