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

Update message #30

Merged
merged 2 commits into from
Dec 25, 2023
Merged

Update message #30

merged 2 commits into from
Dec 25, 2023

Conversation

agoscinski
Copy link
Collaborator

@agoscinski agoscinski commented Sep 8, 2023

Reopen of #26 (comment) because I accidentally push main to the branch and not the HEAD

  • implement AssertResult that allows formatting of assert
    results
  • use answer_key as CheckableWidget name
  • handle error during check more transparent to student by catching
    errors during fingerprint function and asserts and embedding them into
    the assert message
  • implement input parameters for Checks
    • suppress_fingerprint_asserts: specifies if the assert messages that
      use the fingerprint function result are suppressed
    • stop_on_assert_error_raised: Specifies if running the asserts is
      stopped as soon as an error is raised in an assert
  • format check results more structured

📚 Documentation preview 📚: https://scicode-widgets--30.org.readthedocs.build/en/30/

DivyaSuman14 and others added 2 commits December 25, 2023 18:23
* implement AssertResult that allows formatting of assert
  results
* use answer_key as CheckableWidget name
* handle error during check more transparent to student by catching
  errors during fingerprint function and asserts and embedding them into
  the assert message
* implement input parameters for Checks
  - suppress_fingerprint_asserts: specifies if the assert messages that
    use the fingerprint function result are suppressed
  - stop_on_assert_error_raised: Specifies if running the asserts is
    stopped as soon as an error is raised in an assert
* format check results more structured
@agoscinski agoscinski marked this pull request as ready for review December 25, 2023 17:29
@agoscinski agoscinski merged commit 11c7e79 into main Dec 25, 2023
3 checks passed
@agoscinski agoscinski deleted the update_message branch December 25, 2023 17:39
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