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

feat: FSM Builder #9799

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from
Draft

feat: FSM Builder #9799

wants to merge 7 commits into from

Conversation

eliotwrobson
Copy link
Collaborator

@eliotwrobson eliotwrobson commented May 1, 2024

Work-in-progress remake of #6604. Will be incorporating some new features before even taking this out of draft, so no need to review yet. In addition, we almost certainly don't want to merge before the paper for this appears and we can get some feedback from users.

The items to add (so far) are:

  • Fix all javascript linting errors on main file for this element.
  • Add tests for (most aspects of) the grading algorithm.
  • Setting and resetting a non-empty initial configuration. This makes it easy to write questions where students have to fix an incorrect answer.
  • Make grading algorithm more efficient by using wall-following on difference DFAs instead of brute-force check.
  • Update arXiv version of paper with new installation instructions and changes made to grading algorithms (including proof of partial credit).

@eliotwrobson eliotwrobson added under development This PR is under development and shouldn’t be merged yet element Anything related to elements labels May 1, 2024
@eliotwrobson eliotwrobson self-assigned this May 1, 2024
Copy link
Contributor

github-actions bot commented May 1, 2024

All images

Image Platform Old Size New Size Change
prairielearn/executor:1d24f64cef8e9e36fa838f4a536eacc4f2dbb660 null 1627.45 MB 1627.84 MB 0.02%
prairielearn/plbase:1d24f64cef8e9e36fa838f4a536eacc4f2dbb660 linux/amd64 1437.84 MB 1438.21 MB 0.03%
prairielearn/prairielearn:1d24f64cef8e9e36fa838f4a536eacc4f2dbb660 linux/amd64 1627.45 MB 1627.84 MB 0.02%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
element Anything related to elements under development This PR is under development and shouldn’t be merged yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant