#5994: singular.version() yields an error when first called, has no doctest, and
has a strange output imo
---------------------------------------------+--------------------------
Reporter: SimonKing | Owner: was
Type: defect | Status: needs_info
Priority: major | Milestone: sage-6.4
Component: interfaces | Resolution:
Keywords: singular version help.cnf | Merged in:
Authors: William Stein | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
Dependencies: | Stopgaps:
---------------------------------------------+--------------------------
Comment (by SimonKing):
To repeat my questions:
- How important is it to have doc tests that remain valid if the Singular
version changes?
My answer: We could at least have a test marked random. In that way, we at
least make sure that there is no error raised.
- Do we take the patch level version number of Singular serious?
Respectively do we really want the current behaviour (singular.version()
returns a lengthy string), or would we prefer to have it return a tuple of
integers?
My answer: We have
{{{
sage: pari.version()
[2, 8, 0]
sage: r.version()
((3, 2, 1), 'R version 3.2.1 (2015-06-18)')
sage: gap.version()
'4.7.8'
sage: sage0.version()
'SageMath Version 6.9.beta3, Release Date: 2015-08-21'
}}}
Hence, the different interfaces have all kind of different output. Thus,
the current behaviour should be fine, but the following output
{{{
sage: singular.eval('system("version")')
'3170'
}}}
would be equally fine.
My suggestion is: Use the SHORT version string by default, but provide
singular_version and singular.version with an optional "verbose=False"
argument, that allows to obtain the lengthy version string. And of course
fix the error.
--
Ticket URL: <http://trac.sagemath.org/ticket/5994#comment:24>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
