-
Notifications
You must be signed in to change notification settings - Fork 35
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
chores: refactor + cleanup #52
Conversation
Also implemented a `Display` for Lagrange basis polynomials.
60dbadb
to
5fa09c4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very clean PR nice work, well documented too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
scrub a dub dub, nice cleanup!
I could add the crate attribute to enforce docs or throw clippy warning. I think this would be good so we have to document everything including crate/module level and not just body level. |
Working to:
Added
dev-dependencies
:rstest
.polynomial
module (although it may be a bit overkill).pretty_assertions
: