bugDenemo - Bugs: bug #33562, LilyPond parser failures

 
 

bug #33562: LilyPond parser failures

Submitter:  Richard Shann <rshann>
Submitted:  Tue 14 Jun 2011 09:42:12 AM UTC
Votes:  10  
 
Category:  None Severity:  3 - Normal
Item Group:  None Status:  None
Privacy:  Public Assigned to:  None
Originator Name:  Open/Closed:  Open
* Mandatory Fields

Add a New Comment (Rich Markup)
   

Comment Type & Canned Response:
       No canned response available

 

Fri 19 Aug 2011 07:34:13 AM UTC, comment #2: 

One thing that is useful to know to work on the lilypond importer is that the code can be run outside of Denemo (much quicker for testing). You make the directory holding the parser the current working directory and put your test lilypond in a file called test.ly there (IIRC) and then launch the parser. It tests for whether it is running under Denemo and if not opens test.ly and parses that.
You will need to be able to read scheme and to have some understanding of lex/yacc lexer/parsers to make progress. What needs doing is, in itself, fairly simple (accepting embedded scheme in the .ly file - that is stuff starting with # would require a neat little call to scheme read procedure, but the rest is just making more rules like ones already there).

Richard Shann <rshann>
Project Administrator
Thu 18 Aug 2011 10:52:36 PM UTC, comment #1: 

I'm finding similar problems importing simple things like _"4" for a fingering instruction, or \context Staff = "1" gives the importer fits.

If I knew Scheme in the least, I would try to make it tolerant of errors and either skip over them or make some sort of remark in the score to the effect of "something bad happened."  I'll fiddle anyway but I don't anticipate it'll do any good.

Paul Kronenwetter N2KIQ <kronenpj>
Tue 14 Jun 2011 09:42:12 AM UTC, original submission:  

When converting from mxml (with Debian Stable's LilyPond) our lilypond parser failed on

\numericTimeSignature\time 4/4

and

\tempo 4=140

both of these could have been ignored, but instead they aborted the whole process.

In addition, empty staffs were generated (a known problem I think that we could just fix after the parse).

Richard Shann <rshann>
Project Administrator

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by kronenpj (Posted a comment)
  • -email is unavailable- added by kronenpj (Voted in favor of this item)
  • -email is unavailable- added by rshann (Submitted the item)
  •  

    There are 10 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

     

     

     

    Follows 1 latest change.

    Date Changed by Updated Field Previous Value => Replaced by
    2011-08-18 kronenpj Carbon-Copy- Added kronenpj

    Back to the top


    Powered by Savane 3.9