Skip to content
This repository was archived by the owner on Jun 30, 2024. It is now read-only.

Add: Dynamic problems for fitb questions. - #1480

Closed
bjones1 wants to merge 1 commit into
RunestoneInteractive:masterfrom
bjones1:dynamic_problems
Closed

Add: Dynamic problems for fitb questions.#1480
bjones1 wants to merge 1 commit into
RunestoneInteractive:masterfrom
bjones1:dynamic_problems

Conversation

@bjones1

Copy link
Copy Markdown
Contributor

Depends on RunestoneInteractive/RunestoneComponents#1029. A MVP for dynamic problems.

.. fillintheblank:: fitb_dynamic
:dynamic:
a = random.randrange(0, 10)
b = random.randrange(0, 10)
What is {{=a}} + {{=b}}?
- :int(ans) == a + b: Correct!
:int(ans) == a - b: That's subtraction.
:int(ans) == a * b: That's multiplication.
:float(ans) == approx(a / b): That's division.
:x: I don't know what you're doing.

@bjones1
bjones1force-pushed the dynamic_problems branch 3 times, most recently from db1aa74 to 2744e8aCompareJuly 16, 2020 21:49
@bjones1

Copy link
Copy Markdown
ContributorAuthor

Note that these test failures result from not having the updated Runestone components -- templated items such as {{=a}} raise an exception, since the components doesn't store source code for calculating a. All tests pass with the updated components.

@github-actions

Copy link
Copy Markdown

Stale pull request message

@github-actions

Copy link
Copy Markdown

Stale pull request message

@bjones1

Copy link
Copy Markdown
ContributorAuthor

I'd actually like to re-consider this after the last PR for improved testing is merged.

@github-actions

Copy link
Copy Markdown

Stale pull request message

@bjones1

Copy link
Copy Markdown
ContributorAuthor

Let's leave this closed. If RunestoneInteractive/RunestoneComponents#1225 is merged, then I'll rewrite this for the bookserver.

@bjones1
bjones1 deleted the dynamic_problems branch September 20, 2021 14:36
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@bjones1@bnmnetp