New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Deprecated LOM method in max.ts #155
Comments
wow this is a great find, thank you for noting it here.. i havent moved to 11. i ll get this in soon(ish) i have begun some refactoring and documentation work. I ll pick this up. |
Not that Im being paid by Ableton (I wish I was), but 11 is great, highly recommend. |
Maybe its time to retire the scribble.max method alltogether. It is possible to extend Scribbleformax in the following way:
I have recently written this solution for a project of mine, dealing with all the new Live 11 methods and backwards compatibility issues. I can open a PR and extend latest version of Scribbleformax , if you are interested @walmik |
Sounds great @JanZaion ! |
Cool, Ill get to it sometimes mid Jully |
Hi @JanZaion I was renaming a few things
Everything worked fine but as soon as i bring in the new version of Scribbletune (v4.1.0), i m facing a weird issue. It just crashes Live anytime I use webpack to build the updated dist/api.js file :/ In v4.1.0 all the scales and chords were abstracted out to a new module I created and published called harmonics but the rest of the code was more or less the same. I wonder if I ve missed something (like maybe defining the updated version of scribbletune in some max project file) Do you have some idea what could be going on? I suspect it throws an Uncaught ReferenceError: LiveAPI is not defined (but there s no way to validate this as Max and Live, both crash and I havent figured a way to get to some error logs from Max) I ve updated the repo with using v4.1.0 of Scribbletune but I havent commited the breaking api.js. The earlier templateAPIPass is still retained for posterity. If you could check out the latest scribbleformax and then do npm install followed by npx webpack, you can reproduce the issue.
I d love to explore this ^ idea. Would be great if you could give me some example for this, I d like to give it a shot if you re still busy with stuff. Many thanks in advance! |
Hi @walmik I haven' reproduced the error you are facing. But I faced similar issue few months back. I was trying to push Tonal into Max js object with Webpack and it kept crashing the whole thing just like you describe. Huge headache, I wasn't even able to produce an error message. IMHO we are just lucky that the transpiled version of Scribbletune worked up till this point. Hopefully the new api script will retire the need for transpiled js objects in Max altogether.
I was planning on opening a proper PR sometimes later this week, but since you are eager to get going, I have forked the latest version of Scribbleformax and updated Riff with the new api script. There are some notes on this in the PR. I hope you find it usefull, let me know if you need me to clarify something :) |
Thank you @JanZaion I think this is the right way forward. I ll take a look at the PR in scribbleformax and will continue this discussion there. |
Hello @walmik,
I believe there is a deprecated method in max.ts file. Anytime I load a device that uses scribble.max method in Live 11, it throws this message:
I think that the
'remove_notes'
method in max.ts should be changed to'remove_notes_extended'
, since the LOM documentation sais the following:"Deprecated since Live 11.0. Please use remove_notes_extended instead. If a device uses the deprecated method in Live 11, a warning pop-up will notify users that the device needs to be updated."
Simply replacing the method is not backwards compatible with Live 10, so I wrote the following 2 lines:
I would have opened a PR, but I have very little experience writing TS, so Im posting it here. I have tested the solution manually with js object in Max and ES15 syntax. Im posting the ES15 code for posterity:
The text was updated successfully, but these errors were encountered: