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

Add Antidiff and AddConst answer tests #1398

Merged
merged 27 commits into from
Feb 18, 2025
Merged

Add Antidiff and AddConst answer tests #1398

merged 27 commits into from
Feb 18, 2025

Conversation

anst-i
Copy link
Collaborator

@anst-i anst-i commented Feb 15, 2025

Provide ATAntidiff and ATAddConst calculus answer tests, with documentation

anst-i and others added 27 commits May 21, 2024 17:50
It does similar things as ATInt, but it really only checks if the derivatives of the student's answer and the teacher's answer are algebraically equivalent.

Answer test fixtures have been added already. Some of them fail:
- The first one fails due to the additive constant, something to be discussed
- All others fail due to absolute values in logarithms. This test is not designed to account for them, but I currently left them in the answer tests to see the effects. This part is work in progress
Add Antidiff/AddConst to calculus answer test list
WIP: Added short of ATAntidiff
Completed documentation of Antidiff & AddConst answer tests
Clarified some mechanisms, added NONSTRICT explanation
Fixed typos
@anst-i anst-i added this to the 4.9.0 milestone Feb 15, 2025
@anst-i anst-i requested a review from sangwinc February 15, 2025 10:49
@@ -102,6 +102,37 @@ debug:true;
/* [wxMaxima: input end ] */


/* [wxMaxima: input start ] */
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like you saved the sandbox in your development area! Best to copy this somewhere outside the git repro to avoid committing working to the clean distro version.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll fix it!

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! And yes, that's a good idea.

@sangwinc
Copy link
Member

Thanks @anst-i that's a really helpful contribution.

@sangwinc sangwinc merged commit ce0dd7f into dev Feb 18, 2025
11 of 12 checks passed
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