CSE Machine : Fixes Cache and Arrow displacement errors#3650
Merged
martin-henz merged 4 commits intosource-academy:masterfrom Mar 13, 2026
Merged
CSE Machine : Fixes Cache and Arrow displacement errors#3650martin-henz merged 4 commits intosource-academy:masterfrom
martin-henz merged 4 commits intosource-academy:masterfrom
Conversation
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.
Description
This PR targets many of the MUCH-awaited changes in the CSE Machine after the Layout (@gigopogo, @ThatLi) team's PR at #3594 got merged. This introduced many new errors (stated in the PR as well as below here).

The issues stated '2' has been fixed partially with #3627 with the only problem being that the cache was not properly updated during the start of the program leading to only one out of the two new split-cache implementation for the Normal mode vs Printable mode. This has been fixed in this PR.
Additionally, the issue stated '4' was also fixed at #3592.
With this current PR, another issue that is being fixed is the one stated at '1' in the image.
Type of change
How to test
Run any of the following Source codes or one that is mentioned in any of the above mentioned pull requests. The expected behavior should be that on toggling the 'Allignment' operator, the arrows should now be correctly pointing to the frames at their correct new positions. Moreover, switching between Normal and Printable mode should dynamically resize the length of the frames and the gap between them, all while preventing horizontal drifting!
Video.Project.5.mp4
Checklist