-
Notifications
You must be signed in to change notification settings - Fork 163
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
Improve Is{Constant,Univariate}RationalFunction, add tests #2474
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2474 +/- ##
==========================================
+ Coverage 74.06% 74.14% +0.07%
==========================================
Files 484 484
Lines 245498 245545 +47
==========================================
+ Hits 181830 182057 +227
+ Misses 63668 63488 -180
|
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
I just extended this some more, adding helpers for multiplying rational numbers with rational functions over e.g. finite fields (we already have such helpers for addition). |
@fingolfin The helper functions were not added to the HPC-GAP folder. :) |
6550857
to
b3f78f1
Compare
* Change IsConstantRationalFunction and IsUnivariateRationalFunction to return false for objects which are not even rational functions * Improve UNIVARTEST_RATFUN to deal with some more "trivial" cases before giving up * Add convenience helpers for multiplying rational numbers by rational functions over rings which don't contain the rational numbers, complementing the existing helpers doing this for addition * Add tests
return false for objects which are not even rational functions
giving up
functions over rings which don't contain the rational numbers,
complementing the existing helpers doing this for addition