-
Notifications
You must be signed in to change notification settings - Fork 231
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
feat(math): impl remove zeros for sparse univariate poly #430
feat(math): impl remove zeros for sparse univariate poly #430
Conversation
9a9b1af
to
155f934
Compare
155f934
to
78fd8bd
Compare
29ebb3c
to
23f5160
Compare
23f5160
to
742336a
Compare
tachyon/math/polynomials/univariate/univariate_sparse_polynomial_unittest.cc
Outdated
Show resolved
Hide resolved
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.
LGTM
ae88fb4
to
9d78494
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.
LGTM
tachyon/math/polynomials/univariate/univariate_dense_coefficients.h
Outdated
Show resolved
Hide resolved
9d78494
to
4394f2c
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.
LGTM
tachyon/math/polynomials/univariate/univariate_sparse_polynomial_unittest.cc
Outdated
Show resolved
Hide resolved
@batzor For the second commit, I can agree for the sparse coefficients but not with the dense coefficients. What's the motivation behind reducing the usage of remove high degree zeros for dense coefficients? |
@TomTaehoonKim Indeed, the |
250246b
to
6561174
Compare
Zero cannot be in c_terms if the input polynomials are clean
6561174
to
65131d2
Compare
@chokobole I reordered the commits as you requested. Hope its more clear now;; |
tachyon/math/polynomials/univariate/univariate_sparse_coefficients.h
Outdated
Show resolved
Hide resolved
65131d2
to
802ef70
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.
LGTM
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.
LGTM
Description
RemoveHighDegreeZeros()
toRemoveZeros()
in univariate spare polynomial