[Rd] qnbinom with small size is slow

Martin Maechler m@ech|er @end|ng |rom @t@t@m@th@ethz@ch
Mon Aug 24 14:25:58 CEST 2020


>>>>> Constantin Ahlmann-Eltze 
>>>>>     on Fri, 21 Aug 2020 11:51:13 +0200 writes:

    > Hi Martin, thanks for verifying. I agree that the
    > Cornish-Fisher seems to struggle with the small size
    > parameters, but I also don't have a good idea how to
    > replace it.

    > But I think fixing do_search() is possible:

Yes,  but your fix was not correct..
(and I used time to find out)

Note that do_search()  is used in all three of

qnbinom()
 qbinom()
  qpois()

If a change was correct and an improvement, it will be an
improvement for all three cases; if it is only sometimes good,
it may also be sometimes bad for the other two cases.

Note that running the checks on the current R sources makes
almost no sense: Of course the new bugs (your: "slow"; mine:
inaccurate when result is quite larger than 0) are *NOT* yet in
the regression tests.

And it's really not worth doing experiments on the cloud ("QI"
is really increasing the climate change, by each time installing
tons (=> bandwidth) and running 1.5 hours ..):

I don't know what big a part of tree is killed by the bandwith
  (==> electricity of internet routers, cooling, ...)
and the 1.5 hour run
  (==> electricity of the CPUs; cooling; ...)
but I'd rather do test such small changes on my small desktop
where I don't need to install anything etc
(*and* I know that the Swiss electricity is almost exclusively
  produced from no-Carbon-emission plants).

--------

I'm busy for another 1-3 days (taking & grading exams),
but really do want to address this myself, rather than trying
these proposals (and having you burn electricity on routers and
CPUs and coolers) ...

Martin



More information about the R-devel mailing list