Skip to content
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

Upgrade shaker generator to support 2.5 #19

Open
wants to merge 51 commits into
base: develop
Choose a base branch
from

Conversation

tjouinrmld
Copy link

  • Quick first implementation of CSL and SSM for Sugarbox
  • Clean up a bit resources and make sure we get Shaker 2.5 running
  • Add ScriptRunner with detail to track what is happening
  • this code could be promote to Core to support native scripting IMHO

First implementation is still having some timing issue (especially Module A, between test 4 and 5). With timing issues, we miss some test image generation for now... (see logs)
Core need to be fixed IMHO

I put it as draft, feedback welcomed !

Thomas GUILLEMIN and others added 30 commits June 7, 2023 18:39
* Fix tape, to be error prone (Tom1975#14)
* Fix recording, using path
* Bugfix/fix tape and record (Tom1975#15)
* Fix: remove accentued characters
* DISABLE despotik desisgn due to non reproductible bug on CI (will be investigated later)
Add custom opcode in Shaker to handle auto printscreen
… short, but this shouldn't cause any trouble)
@Tom1975 Tom1975 marked this pull request as ready for review January 30, 2024 15:39
@tjouinrmld
Copy link
Author

hi @Tom1975
I'm not sure that unit test were working on dev branch prior to my change (dsk issue).
Hope the last commit will fix compilation on Ubuntu !
Thanks

@Tom1975
Copy link
Owner

Tom1975 commented Feb 4, 2024

hi @Tom1975 I'm not sure that unit test were working on dev branch prior to my change (dsk issue). Hope the last commit will fix compilation on Ubuntu ! Thanks

Thanks for the fix for ubuntu, and the pull request.

About the tests, I believe it may be due to some problems with keyboard layout (only file that change).
What's the reasons of the change ? It's a bit big to find out easily :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants