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

Partly revert d53a197 & 36fc539 to workaround #233 #238

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

shhyou
Copy link
Collaborator

@shhyou shhyou commented Dec 16, 2024

A more appropriate fix should be applied in the future.

To workaround #233, this partly reverts commits

  • "highlight universe when set-up fails at run-time". d53a197
  • "fix resource administration in run and launch-many-worlds". 36fc539

Otherwise, when the object [((new-world (if #,rec? aworld% world%)) w #,@args)] and [((new-universe universe%) u #,@args)] are created in the main thread, the continuation marks of the stepper are somehow broken.

The two follow up commits forward the check-with exceptions from (o) in esp/thd to the main thread and repackages it with big-bang's stack frames.

To workaround racket#233, this partly reverts commits

- "highlight universe when set-up fails at run-time".
   d53a197
- "fix resource administration in run and launch-many-worlds".
   36fc539

Otherwise, when the object `[((new-world (if #,rec? aworld% world%)) w #,@Args)]`
and `[((new-universe universe%) u #,@Args)]` are created in the main thread, the
continuation marks of the stepper are somehow broken.

A more appropriate fix should be applied in the future.
@shhyou shhyou added stepper teachpack [2]htdp teachpack labels Dec 16, 2024
@shhyou shhyou marked this pull request as draft December 28, 2024 20:46
In big-bang tests, if the tick frequency is high, some to-draw steps
(e.g. the one immediately following the launch of big-bang) can be
absent in stepper because no such call are made.

This commit adds the `(repetition ,lo ,hi ,steps) form to stepper tests to
account for skippable steps.
@shhyou shhyou marked this pull request as ready for review January 1, 2025 04:58
@shhyou shhyou requested a review from jbclements January 1, 2025 18:02
@shhyou
Copy link
Collaborator Author

shhyou commented Jan 1, 2025

@jbclements This change is intended to be merged after the branch day.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stepper teachpack [2]htdp teachpack
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant