You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We already discussed this before, just to make sure it is not forgotten:
When possible we should consider unpacking all UPoly into MPolyRingElem then working with those, then only wrapping the result back into UPoly. That should avoid redundant allocations and also in some cases ensure we use optimized algorithms.
Caveat: we need to make sure the unwrapped polynomials all have the same parent. There is no nice helper for this right now, but there should be... in the meantime we can write something like this (untested, may need tweaking, but gives the right idea):
functionunwrap(a)
R =parent(a)
s =data(a)
ifnvars(parent(s)) !=nvars(R)
s =upgrade(R, s)
endreturn s
end
The text was updated successfully, but these errors were encountered:
We already discussed this before, just to make sure it is not forgotten:
When possible we should consider unpacking all
UPoly
intoMPolyRingElem
then working with those, then only wrapping the result back intoUPoly
. That should avoid redundant allocations and also in some cases ensure we use optimized algorithms.Caveat: we need to make sure the unwrapped polynomials all have the same parent. There is no nice helper for this right now, but there should be... in the meantime we can write something like this (untested, may need tweaking, but gives the right idea):
The text was updated successfully, but these errors were encountered: