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

Added tests for power and log + unchained suite #8

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

arkanoid87
Copy link

as I failed generate meaningful fixes today, I've generated a longer test suite, that includes "pure" tests for power and log, and "unchained" version for all "pure" tests. I made this to get a better picture of what is currently working ad what is not at the moment. I hope this helps.

as I failed generate meaningful fixes today, I've generated a longer test suite, that includes "pure" tests for power and log, and "unchained" version for all "pure" tests.
I made this to get a better picture of what is currently working ad what is not at the moment. I hope this helps.
How to deal with decimal exponents?
@arkanoid87
Copy link
Author

I'm trying some fixes in my fork

I fixed power & exponentials when not using unchained (comparing results with vs python uncertainties), but how to deal with float exponents?

@Vindaar
Copy link
Member

Vindaar commented Nov 16, 2022

If you feel like some of these tests are still useful now that #9 is merged, please feel free to rebase and create a new PR for them or rebase & force push this one.

@arkanoid87
Copy link
Author

plenty of changes! I'll focus on testing, then rebase if necessary

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