Fix typos, and make Unions report more uncombined sets #306
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This fixes a couple of typos in
AnswerCheckers.pm
, makes sure that the derivative of a fraction copies the denominator when squaring it (otherwise the derivative shares a tree node with the original), and fixes an error inisReduced()
for Union objects where unions like[0,1) U {1}
would be reported as reduced (rather than requiring it to be[0,1]
).The latter can be checked using
With the patch, this should report "Reduced = 0, Reason = uncombined sets", while without the patch, it should be "Reduced = 1, Reason = " with no reason.