all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Ricardo Wurmus <rekado@elephly.net>
To: zimoun <zimon.toutoune@gmail.com>
Cc: Guix Devel <guix-devel@gnu.org>
Subject: Re: Move r-{desolve, quadprog, pracma, subplex} from maths.scm to cran.scm
Date: Tue, 10 Dec 2019 16:38:29 +0100	[thread overview]
Message-ID: <87d0cw5gy2.fsf@elephly.net> (raw)
In-Reply-To: <CAJ3okZ3qyfGFmjtXzfXXxi6qQ1J4KZ7WHQbO1Y6PnQfZKZTtXg@mail.gmail.com>


zimoun <zimon.toutoune@gmail.com> writes:

> If I understand well, the policy is: the packages in the file cran.scm
> cannot import '(gnu packages bioconductor).

Yes.  It’s not official policy, but I think we should avoid mutually
recursive module imports when we have a choice.

> In the file cran.scm, for example the packages r-shiny or r-sankeyd3
> do not come from CRAN but directly from Github.
> Maybe, we could group all the 6 non-CRAN packages and the 4 CRAN
> packages depending on Bioconductor to a unique file.
>
> What do you think?

Yes, the should probably be moved.  It’s also very easy to miss them
when doing mass upgrades, hidden among all these CRAN packages.

>> > 2.
>> > The second point is that the file statistics.scm contains 227 packages
>> > and 206 are cran-uri. And the file cran.scm already contains 602
>> > packages, so it will be almost as python-xyz.scm is. Why not. Then,
>> > this file statistics.scm will be almost empty and I propose instead to
>> > populate the files maths.scm, python-xyz.scm.
>> >
>> > What do you think?
>>
>> statistics.scm was the only thing we had for R in the beginning.  When
>> it grew and it became apparent that more and more CRAN packages would be
>> added, I felt that the module became less suitable.
>
> Sorry, my words was not good enough. I propose to remove the file
> statistics.scm by reordering the packages.

I think it’s fine to keep statistics.scm even when it’s small.  R and
pspp *are* stats packages after all, so this module remains the perfect
location for them.

>> I don’t understand your comment about “maths.scm” and “python-xyz.scm”.
>> I think it’s good to move CRAN and Bioconductor packages out of
>> statistics.scm whenever possible.  We can leave the actual statistics
>> programs there as well as potentially unmovable CRAN packages.
>
> In the file statistics.scm, once move to cran.scm or bioconductor.scm,
> there is few packages. For example 2 packages coming from PyPI and I
> propose to move them to the file python-xyz.scm. The other ones could
> be moved to the file maths.scm or the unamed (yet) file (see above).

I’d like to avoid moving anything *to* python-xyz.scm, because it’s a
bit of a dump.  I think it’s fine to just focus on the R packages for
now and address the rest later — if at all.

--
Ricardo

      reply	other threads:[~2019-12-10 15:38 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-31 15:37 Move r-{desolve,quadprog,pracma,subplex} from maths.scm to cran.scm zimoun
2019-11-07 19:08 ` Move r-{desolve, quadprog, pracma, subplex} " zimoun
2019-11-07 22:42   ` Ricardo Wurmus
2019-11-22 14:47 ` Ricardo Wurmus
2019-11-22 16:03   ` zimoun
2019-12-09 23:25   ` zimoun
2019-12-10  8:41     ` Ricardo Wurmus
2019-12-10 12:17       ` zimoun
2019-12-10 15:38         ` Ricardo Wurmus [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87d0cw5gy2.fsf@elephly.net \
    --to=rekado@elephly.net \
    --cc=guix-devel@gnu.org \
    --cc=zimon.toutoune@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.