On 7/27/19 6:32 PM, Waldek Hebisch wrote: > oldk1331 wrote: >> The second patch uses "construct" to greatly simplify "*" in MRING. > > When M is ordered old code tries to minimize need for sorting. > It is hard to see if this really gains some speed but it > would be better to do some measurements and compare speed > of old code and your new one.
You are right, old code is about 4 times faster than mine for MRING(INT,FreeMonoid(INT)). (after work around the compiler bug in 'concat!') So I'll change for "not (M has OrderedMonoid)" only. >> The third patch does some cleanup for MRING. What's your opinion on the third patch? -- You received this message because you are subscribed to the Google Groups "FriCAS - computer algebra system" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/fricas-devel/CAGBJN91ohV5woUPaNUW7fp1e%3DPVUPbpc0DrDfrG%2B3p%2BgDe4wbw%40mail.gmail.com.
