Re: [fricas-devel] patch to recent commit r2473

2018-06-25 Thread Waldek Hebisch
oldk1331 wrote: > > On Sun, Jun 24, 2018 at 10:42 PM, Waldek Hebisch > wrote: > > oldk1331 wrote: > >> > >> But for 'min'/'max', the 'map' paradigm is not very suitable. > >> The appropriate way is to use 'reduce'. > > > > Well, one can use imperative 'map' to implement 'reduce'. > > The point is

Re: [fricas-devel] patch to recent commit r2473

2018-06-24 Thread oldk1331
On Sun, Jun 24, 2018 at 10:42 PM, Waldek Hebisch wrote: > oldk1331 wrote: >> >> But for 'min'/'max', the 'map' paradigm is not very suitable. >> The appropriate way is to use 'reduce'. > > Well, one can use imperative 'map' to implement 'reduce'. > The point is that 'map' is the only place which _

Re: [fricas-devel] patch to recent commit r2473

2018-06-24 Thread Waldek Hebisch
oldk1331 wrote: > > But for 'min'/'max', the 'map' paradigm is not very suitable. > The appropriate way is to use 'reduce'. Well, one can use imperative 'map' to implement 'reduce'. The point is that 'map' is the only place which _must_ know how to iterate over an aggregate. Given 'map' all othe

Re: [fricas-devel] patch to recent commit r2473

2018-06-23 Thread Bill Page
On Sat, Jun 23, 2018 at 4:52 AM, oldk1331 wrote: > But for 'min'/'max', the 'map' paradigm is not very suitable. > The appropriate way is to use 'reduce'. > +1 -- You received this message because you are subscribed to the Google Groups "FriCAS - computer algebra system" group. To unsubscribe

Re: [fricas-devel] patch to recent commit r2473

2018-06-23 Thread oldk1331
But for 'min'/'max', the 'map' paradigm is not very suitable. The appropriate way is to use 'reduce'. Can I commit my original patch? On Sat, Jun 23, 2018 at 12:09 AM, Waldek Hebisch wrote: > oldk1331 wrote: >> >> fix documentation and use a simpler implementation > > Documentation parts is obvi

Re: [fricas-devel] patch to recent commit r2473

2018-06-22 Thread Waldek Hebisch
oldk1331 wrote: > > fix documentation and use a simpler implementation Documentation parts is obvious (OK). Concerning implementation my idea is to eventually have map1 exported (probably as 'map'). Trying to do this caused problems with overload resolution, so for now it is local. However, I h

[fricas-devel] patch to recent commit r2473

2018-06-19 Thread oldk1331
fix documentation and use a simpler implementation diff --git a/src/algebra/aggcat.spad b/src/algebra/aggcat.spad index 0a844a5b..8261664f 100644 --- a/src/algebra/aggcat.spad +++ b/src/algebra/aggcat.spad @@ -120,11 +120,11 @@ ++ For multisets \spadfun{members} gives result with no repet