Skip to content

Upgrade shaker generator to support 2.5#19

Open
tjouinrmld wants to merge 51 commits into
Tom1975:developfrom
tjouinrmld:feature/shaker2.5
Open

Upgrade shaker generator to support 2.5#19
tjouinrmld wants to merge 51 commits into
Tom1975:developfrom
tjouinrmld:feature/shaker2.5

Conversation

@tjouinrmld
Copy link
Copy Markdown

  • 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
Copy Markdown
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
Copy Markdown
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