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

Blocks language supports benchmark program #513

Merged
merged 4 commits into from
Mar 6, 2024

Conversation

asolove
Copy link
Contributor

@asolove asolove commented Mar 5, 2024

This PR contains changes to the Snap definitions of the Pyret language to enable us to construct the demo program we want to be able to run the benchmark program.

In previous testing, we found a few missing items:

  • Introduce a new variable and assign it to a value: var = val

  • Lookup for Row, StringDict and other map-like things: map[key]
    image

  • image-scatter-plot (turns out this already worked and Adam just misunderstood)

  • load-table can define multiple column names as variables
    image

  • allow multiple arguments in function definitions and applications
    image

@asolove asolove changed the title [WIP] Blocks: language definition edits Blocks language supports benchmark program Mar 6, 2024
@asolove asolove marked this pull request as ready for review March 6, 2024 12:14
@asolove
Copy link
Contributor Author

asolove commented Mar 6, 2024

@jpolitz I think this is everything needed to run our benchmark program. There are only changes to the blocks definitions. It would be helpful if we could merge this to horizon and deploy. Then we can test it out in an environment that can use context shared-gdrive.

@jpolitz jpolitz merged commit 072561d into brownplt:horizon Mar 6, 2024
1 check passed
@jpolitz
Copy link
Member

jpolitz commented Mar 6, 2024

I believe this is now live at https://pyret-horizon.herokuapp.com/blocks

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