CoderLine / alphaTab

alphaTab is a cross platform music notation and guitar tablature rendering library.
http://www.alphatab.net
Mozilla Public License 2.0
1.26k stars 202 forks source link

GPX issues #91

Closed mbektimirov closed 8 years ago

mbektimirov commented 8 years ago

Hi Daniel, I've found some critical gpx reading issues and listed it below. I will continue to add it in this thread if there will be some more bugs.

Danielku15 commented 8 years ago

I fixed the 2 rendering issues you mentioned. the first was related to arpeggio rendering. I mixed the y coordinate with the scale which lead to this huge wave symbols being rendered :sweat_smile:

The whammy reading was really easy to fix. There was a break missing. I'm not yet sure whar the WhammyBarExtend exactly means so I simply ignore it.

Regarding the track rendering I just got a question regarding this via mail and fixed it already. The bug was related to some missing call to the accidental helper which does the actual note line calculation.

Feel free to reopen the issue if you discover more issues.

mbektimirov commented 8 years ago

The problem is still the same with drum track and Metallica tab. It breaks on StaveGroup.BuildBoundingsLookup where firstStaveInAccolade is null when I'm trying to render the drums track.

mbektimirov commented 8 years ago

I think it is a gpx parser bug because drums are not playing at all in this tab.

Danielku15 commented 8 years ago

The drum track renders fine on my machine but yes it's not playing. I'll check what happens there.

Danielku15 commented 8 years ago

I added proper parsing of the element/variation tags that are used for drumkits on Guitar Pro 6. This way the drum tabs are properly rendered and played.

mbektimirov commented 8 years ago

Thanks, it works perfect!