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

Stepper: Import statements cause errors #1223

Closed
leeyi45 opened this issue Feb 27, 2022 · 2 comments · Fixed by #1453
Closed

Stepper: Import statements cause errors #1223

leeyi45 opened this issue Feb 27, 2022 · 2 comments · Fixed by #1453
Assignees
Labels
Bug Something isn't working

Comments

@leeyi45
Copy link
Contributor

leeyi45 commented Feb 27, 2022

Running any code with an import statement in the stepper simply produces the output error

image

@leeyi45 leeyi45 added the Bug Something isn't working label Feb 27, 2022
@martin-henz martin-henz changed the title Import statements cause errors in the stepper Stepper: Import statements cause errors Feb 28, 2022
@martin-henz
Copy link
Member

martin-henz commented Aug 23, 2022

Instead of an error, the stepper should accommodate imported modules and display values created by modules in appropriate ways. For example, runes should be displayed within steps of the stepper:

import { heart, quarter_turn_left, show } from "rune"; show(quarter_turn_left(heart));
=>
show(quarter_turn_left(■));
=>
show(■);
=>
■;

In this case, the stepper would use the ■ symbol to represent any rune.

Of course even better would be to display a mini-version of the actual rune in the stepper:

import { heart, stack, show, red } from "rune"; show(quarter_turn_left(heart));
=>
show(quarter_turn_left(♥));
=>
show(❥);
=>
❥;

@martin-henz
Copy link
Member

In addition, every evaluation of "show" should lead to a display of the rune in the runes tab. This is already working in the environment visualizer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
5 participants