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 a Poly wrapper struct, but don't change any operations yet #27

Merged
merged 8 commits into from
Apr 8, 2024

Conversation

teor2345
Copy link
Collaborator

@teor2345 teor2345 commented Apr 8, 2024

This PR creates a wrapper struct Poly and forwards existing operations to it, without changing their implementations yet. (That can happen in future PRs.)

The "boring" implementations that just forward to the underlying DensePolynomial have their own module file.

Part of #13.

@teor2345 teor2345 added the enhancement New feature or request label Apr 8, 2024
@teor2345 teor2345 self-assigned this Apr 8, 2024
@teor2345 teor2345 changed the title 3. Add a Poly wrapper struct, but don't change any operations yet Add a Poly wrapper struct, but don't change any operations yet Apr 8, 2024
@teor2345
Copy link
Collaborator Author

teor2345 commented Apr 8, 2024

Rebased onto #25 + #26 + main

Base automatically changed from more-lints to main April 8, 2024 20:12
@teor2345 teor2345 merged commit 4ecdd04 into main Apr 8, 2024
20 checks passed
@teor2345 teor2345 deleted the wrap-poly branch April 8, 2024 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants