--
You received this message because you are subscribed to the Google Groups "sage-devel" group.
To unsubscribe from this group and stop receiving emails from it, send an email to sage-devel+...@googlegroups.com.
To post to this group, send email to sage-...@googlegroups.com.
Visit this group at http://groups.google.com/group/sage-devel.
For more options, visit https://groups.google.com/d/optout.
On 2015-09-09 10:50, John Cremona wrote:
Re (B): a deprecation warning is normally a message to the user that
they should start to do things in a different way, as the old way will
stop working. This does not seem appropriate here, since it is not the
user who is expected to change behaviour (unless you regard all users as
developers!). So rather than a deprecation warning as such why not a
message saying
"OLDSTYLEPACKAGE is no longer supported by SageMath; if it was
important for your work, please contact <sage-support?> with a request
to upgrade the package."
I think the wording is just a detail and not relevant for this vote, so you can think of option (B) as: display some kind of message and just install the package.
Anyway, the proposed wording in #19158 is
You are about to download and install an old-style package. While this might still work fine, old-style packages are unmaintained and deprecated.
This package will be removed in future versions of SageMath. If you care about this package, you should make a proper new-style package instead. For more information about making Sage packages, see
http://doc.sagemath.org/html/en/developer/packaging.html
But even if there is just a single old-style package
which works and is useful
How does option (D) make "Sage accessible to 100 other users" compared
with options (B) or (C)?
There are other solutions to that problem, any of the following will do:
* a proper implementation of (B) to make them figure it out
* the proposed #19105
* removing the old python spkg from the server
But by reverting #19004 (which is essentially what #19158 does), it is
also possible to install them using "sage -i <package>".
But by reverting #19004 (which is essentially what #19158 does), it is
also possible to install them using "sage -i <package>".It is possible to try, and almost always fail miserably, to install them.
To the discussion at hand, I think the current system of not allowing old spkgs to be installed is a waste. What are the current old style (non-experimental) spkgs, which of them have you tried, and which of them don't work on what system. I'm willing to put in some time and effort to convert some of them to new style (specifically chomp and Simon's cohomology) to try and get this resolved. However, if we are going to go with the current route, we should have a major version number bump for the next stable release.
TOPCOM vs. topcom seems very unlikely to cause a Sage installation to self-destruct to me
For the record: A deprecation warning does *not* mean that the user is
about to do something stupid, but that (s)he is doing something that used
to be supported but soon will not be.
However, I believe that "sage -i <full url of an old style spkg>" should
still be supported.
On Wednesday, September 9, 2015 at 11:56:28 AM UTC+2, Jeroen Demeyer wrote:But even if there is just a single old-style package
which works and is useful
I disagree with that rationale. We want to maximize value for the scientific community. If that means inconveniencing a single user in order to make Sage accessible to 100 other users then we should do that.
On Wednesday, September 9, 2015 at 10:09:21 AM UTC-7, Jeroen Demeyer wrote:On 2015-09-09 17:06, John H Palmieri wrote:
> I think that we should white-list at least some of these, and also try
> to convert some of them (all of them?) to new-style packages. Same with
> Simon's p_group_cohomology package, which didn't build correctly for me.
> I don't have objections to leaving things as they are (i.e., completely
> opaque, maybe also stop them from being listed in "sage --optional") for
> the broken old-style spkgs (many of the others, although I didn't test
> all of the ones that were last modified more than 5 years ago).
OK, I am adding this as a variation on (D):
(D''): sage -i OLDSTYLEPKGNAME should give an error, except if
OLDSTYLEPKGNAME is in a pre-determined white-list of packages.
But the hard question remains: which packages should be white-listed?
From the list I posted, I would suggest
ore_algebra (pretty small, should be easy to convert, I can work on that)
chomp
Also p_group_cohomology, once it's been fixed.
Does anyone want to advocate for any of these?
Are you saying that the fact that something *works*
instead of reading and posting here, I just spent time converting jones_numfield to new style. (#19174)
If every poster here does spend half a day or a day on this, the issue will be moot... :-)
But this argument completely fails when backwards compatibility is
involved. Actively removing a feature (even if broken 99% of the time)
without deprecation is a very exceptional move which should require
exceptional justification.
Is biopython pip-installable?
On 2015-09-09 22:17, Thierry wrote:
> sorry for not participating actively to this long thread, i am currently
> pretty far from computers. Let me just mention that this discussion
> already appears some time ago, and that somehow, the mailing-list is
> probably not the appropriate tool for solving such an issue, while the
> wiki is definitely more appropriate for classifying things, see how big is
> the work to be accomplished, and eventually getting things done. During a
> previous similar thread, i opened the following page, which might a good
> tool if we keep it up-to-date
I never meant this thread to be about
classifying/upgrading/porting/fixing old-style packages, but somehow
many people started thinking in that direction.
This thread was meant to answer the very simple question: what should
happen if a user does
sage -i OLDSTYLEPACKAGE
As long as there is still at least one old-style package left, this
question makes sense and should be answered on the short term before the
next Sage release.
Let's try to summarize what we have so far.
I am going to ignore responses of the form "just port all/some packages to new-style". That's certainly a good thing to do, but it doesn't answer the question. It's also a more long-term solution, while we need something by the next stable Sage release.
So far, nobody was in favour of (A) and (C), so that gives us 2 remaining options:
(B) sage -i OLDSTYLEPKGNAME should still work but with a clear warning message.
(D) sage -i OLDSTYLEPKGNAME should give an error (possibly a more helpful error message than what we currently have).
Votes so far are:
(B): Jeroen Demeyer, Simon King
(D): Volker Braun, John Cremona, John Palmieri
Anybody else wants to give his/her opinion?
Jeroen.
On 2015-09-10, John Cremona <john.c...@gmail.com> wrote:
> Am I right that Simon's package is one (perhaps the only one) for which (B)
> would be useful? If that is true then I don't wish to make things hard for
> Simon. But then I don't know what the obstacles are to converting his spkg.
Partially my ignorance.
Partially the fact that the code is divided into three parts. The first
has a proper upstream (MeatAxe), but we use a very old version (that said,
even the latest version is several years old, and does not appear to be
faster, but has almost all function names changed). The other two parts are
not published elsewhere, and somehow it seems odd to me to artificially
create an upstream repository, as the spkg *IS* the upstream repository
(but perhaps I am the only one who finds it odd...).
Anyway, it currently looks like the spkg will be converted to a
new-style package.
Something else: Of course each package has a maintainer resp. an
upstream contact. Can this information be automatically extracted? Then,
if an error occurs during installation, for whatever reason, the error
message could clearly state: "Please contact f...@bar.com about the
problem." If I recall correctly, the current error messages only
mention sage-devel.
Best regards,
Simon
Votes so far are:
(B): Jeroen Demeyer, Simon King, Sébastien Labbé
(D): Volker Braun, John Cremona, John Palmieri
Anybody else wants to give his/her opinion?
For the record: I think ANY backward incompatible internal change that *can*
be covered by a deprecation warning (thus, changed import locations,
changed function names or argspecs, etc) *has* to be covered by a
deprecation period
I thought some patchbots did test optional packages on a more or less regular
basis?
I thought some patchbots did test optional packages on a more or less regular
basis?No, the build/patchbots only test standard packages. Of course we would be happy to have you volunteer as the person in change of testing all optional packages.
I thought some patchbots did test optional packages on a more or less regular
basis?No, the build/patchbots only test standard packages. Of course we would be happy to have you volunteer as the person in change of testing all optional packages.R has the resources to do this, and is very picky about
their packages (even though they have thousands). The new thing where all installed optional packages are tested is a good start, it's clearly already helped.
--
You received this message because you are subscribed to the Google Groups "sage-devel" group.
To unsubscribe from this group and stop receiving emails from it, send an email to sage-devel+...@googlegroups.com.
To post to this group, send email to sage-...@googlegroups.com.
Visit this group at http://groups.google.com/group/sage-devel.
For more options, visit https://groups.google.com/d/optout.
R has the resources to do this, and is very picky aboutR has vastly more resources than us to put mildly.