helge17 / tuxguitar

Improve TuxGuitar and provide builds
Other
303 stars 29 forks source link

Separated default gmChannels for gpx #396

Closed Lukramancer closed 1 month ago

Lukramancer commented 1 month ago

Recently I've found some .gp files which work fine in GuitarPro but don't in TuxGuitar. Instruments of all tracks were replaced by one of the first track.

I've discovered that they are in GP7 format without specified general midi channels at MidiConnection element in score.gpif file in them.

empty tags

In that case TuxGuitar reads default value 0 as midi channels. The same midi channel in all tracks causes all channels but the first one skip their initialization.

I made sure that the default channel value was one that had not yet been used.

guiv42 commented 1 month ago

I tested with a 14-track .gp song, after manual deletion of the content of all PrimaryChannel and SecondaryChannel nodes:

So, forget my comment and let's keep this PR unchanged.

Lukramancer commented 1 month ago

I'm not expert in GM at all. The thing about maximum channels amount is my oversight. I'll explore that issue more, cause I figured out one more thing. Using default secondary GM channel as 0 can double max instruments available. Can you please share that .gp file for future testing?

guiv42 commented 1 month ago

Can you please share that .gp file for future testing?

Sure. I don't have so many .gp files, and only 1 with enough tracks, you can find it here