Update randVar() #43434

Merged
merged 2 commits into from Mar 17, 2013

Conversation

Projects
None yet
1 participant
Member

petercollingridge commented Mar 17, 2013

Redoing #43394

Ensure exercises use randVar() in math-format (is there a reason for it being in math-format rather than just math?) when generating variable names. Ensure variable names don't include u or v, which could be confused.

Part of me thinks students ought to be able to distinguish u and v, but then again, the exercises are meant to test their mathematical ability, not their ability to read the Latin alphabet.

Deals with #43236 and #43385.

petercollingridge referenced this pull request Mar 17, 2013

Use a less ambiguous letter
fix #43236

Test plan: minimal

Auditors: cbhl

petercollingridge added a commit that referenced this pull request Mar 17, 2013

Merge pull request #43434 from petercollingridge/randvar2
fix #43236; fix #43385;

Test plan: check each exercise still displays an equation

Auditors: beneater, cbhl

@petercollingridge petercollingridge merged commit 1770e95 into Khan:master Mar 17, 2013

@petercollingridge petercollingridge deleted the petercollingridge:randvar2 branch Apr 18, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment