Collaborative project!
-
I've pushed the cleanup for
ScriptProcessorMain.js
. This one was more complicated so please test. -
@d-healey I'm away from my keys. will test itb later :)
-
@hisefilo Cool, if everything turns out to be ok I'll merge it into the master branch
-
@d-healey if console doesn't claim, should be fine I guess. :)
-
@hisefilo The console is fine but I might have messed up some of the pitch/gain/decay values or the interaction between the controls and the modulators - I don't think I have but it's nice to have a second opinion.
-
@d-healey okodoki will tell u later
-
-
@d-healey just a question. Loops at runtime leads to glitches (I remember that on some attempt I did). Don't know if this will do the same
-
@hisefilo All the loops are in deferred callbacks so shouldn't cause any issues.
-
@hisefilo Nope Greg's branch is from today for the preset browser look and feel, I'm just waiting you guys to sort the script cleaning before I merge ;)
-
@ustk ohhh nice :) will require a custom Hise compilation??? at my end?
-
@hisefilo Humm... How old is your version?
The implementation of the LAF is from October, so chances are big you're good with your version
And no, no need to set any flags -
@hisefilo Have you modified the key range?
If it's done so we can tick that and the todo list seems to be empty
Though there's still need to add Lalaland logo -
@ustk 30Jan2020 :)
-
I've just discovered a HUGE sound and playability thing to be improved. Main3 module is doing a weird click due phase shift I did. I will fix that after both merges :)
-
@hisefilo oh... something that is present in the actual master branch?
-
@ustk yes!
-
@ustk it's a super easy fix. don't worry. will do later. saving _local. as reference
-
@hisefilo sure my man
-
I've added the copyright and license notice to each script file. I put both Filo and Ustk as copyright holders on all files. I added myself to the two I re-wrote during the code cleanup yesterday.
I've also added a full license file to the repo, this file is called COPYING which is the standard GNU naming convention. This file should be displayed to the user in the installer as the license they have to agree to before installing the plugin.
All my changes have now been merged and pushed to github.