Repository navigation
Logic EXS24: zones without a sample are left out instead of playing the sample of another zone - #494
Open
douglas-carmichael wants to merge 1 commit into
Open
douglas-carmichael wants to merge 1 commit into
douglas-carmichael wants to merge 1 commit into
Conversation
…he sample of another zone Logic stores a zone which has no sample - an empty zone, which plays nothing - with the sample index -1. Such an index was replaced by the index of the zone, so the zone either played the sample whose number matched it or, behind the end of the sample list, logged 'Sample index out of bounds' once per zone. 11 instruments of the Logic and GarageBand libraries contain empty zones: 'Applause and Laugher' maps its 18 sounds onto 128 zones, one per key, and got a second copy of these sounds on its 18 lowest keys; 'Dry Standard Kit' played crash and click samples on 14 empty velocity layers of E2 and C#2. In Goldbaby's 'KaZioTron Keys', 'Bass_CT310_1_Cass' played its C3 sample on F#1 and G1, where Logic plays the E1 and G#1 zones which were stretched over the empty F#1 zone, and 'Mello_MT70_2_CassDC2' got its 31 organ samples a second time on the keys of its 31 empty noise zones, with their loops. Empty zones are now left out and counted in one message. Only if not a single zone of a file references a sample, the zones still play the samples in their order, as before. Analyse (-ad) compared with the previous commit: the 231 instruments of the BlueWave library are identical, of the 162 of 'KaZioTron Keys' only the 4 with empty zones change, and the 11 instruments of the Logic and GarageBand libraries lose their stray zones and 646 errors.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Logic stores a zone which has no sample - an empty zone, which plays nothing - with the sample index -1. The EXS24 reader replaced such an index with the index of the zone. The zone then either played the sample whose number happened to match, or, when that number was behind the end of the sample list, logged
Sample index out of boundsonce per zone.Empty zones are common in real instruments. 11 instruments of the Logic and GarageBand libraries contain them:
In Goldbaby's 'KaZioTron Keys':
Change
Left out 31 zone(s) without a sample, which play nothing in Logic either.IDS_EXS_EMPTY_ZONES, and a CHANGELOG entry in a new 'Logic EXS24' block of 20.4.0.Tests
Analyse with details (
-ad), compared withmain:Sample index out of boundserrors are gone. 'Dry Standard Kit' has 193 instead of 207 zones.The new 'Logic EXS24' block sits in front of 'SFZ', where #490 and #491 add their 'Elektron Tonverk Preset' block. Whichever is merged second needs both blocks kept.