Skip to content

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

Open
Meratyn wants to merge 2 commits into
Aalto-LeTech:jaal2.0from
Meratyn:jaal2.0
Open

Add the JAAL2.0 working player fixes to the JAAL2.0 branch#16
Meratyn wants to merge 2 commits into
Aalto-LeTech:jaal2.0from
Meratyn:jaal2.0

Conversation

@Meratyn
Copy link
Copy Markdown
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
@atilantera
Copy link
Copy Markdown

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

@atilantera atilantera 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