Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Update randVar() #43394

wants to merge 4 commits into


None yet
1 participant

petercollingridge commented Mar 16, 2013

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 16, 2013

Use a less ambiguous letter
fix #43236

Test plan: minimal

Auditors: cbhl

petercollingridge commented Mar 16, 2013

Hmmm, how did that graph-legend branch get merged into this?

This is annoying. Is there any way I can get rid of that merge without removing all the other commits?

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

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