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

Add the JAAL2.0 working player fixes to the JAAL2.0 branch #16

Open
wants to merge 2 commits into
base: jaal2.0
Choose a base branch
from

Conversation

Meratyn
Copy link
Collaborator

@Meratyn Meratyn commented Sep 16, 2022

The fixes of #13 were added to JAAL1.1 branch, but not JAAL2.0. Now they are added to JAAL2.0 as well.

Export the gradable property from the JSAV in the player test bench as well.

Now it records correct JAAL 2.0.
Correctly set initial state from the submission.
Flatten modelAnswerSteps and filter on gradable property.
@Meratyn Meratyn added the dijkstra-exercise Dijkstra's algorithm exercise label Sep 16, 2022
@Meratyn Meratyn added this to the JAAL 2.0 milestone Sep 16, 2022
@Meratyn Meratyn linked an issue Sep 16, 2022 that may be closed by this pull request
@atilante
Copy link

Somehow I cannot make this work. I will investigate this myself.

@atilante atilante self-assigned this Sep 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dijkstra-exercise Dijkstra's algorithm exercise
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Testbench: DijkstraPE-research recording broken
2 participants