#7496: symbolic variable names should be valid identifiers, or ridiculousness
follows
-----------------------------------+----------------------------------------
Reporter: was | Owner: burcin
Type: defect | Status: needs_info
Priority: major | Milestone: sage-4.7.1
Component: symbolics | Keywords: sd31
Work_issues: | Upstream: N/A
Reviewer: Karl-Dieter Crisman | Author: Volker Braun
Merged: | Dependencies:
-----------------------------------+----------------------------------------
Changes (by kcrisman):
* status: needs_review => needs_info
Comment:
Ok, I think this is still ok, though I am a little concerned about both of
the following being bad:
{{{
sage: var(' x')
(, x)
}}}
not good because an empty string is a variable
{{{
sage: var(' x')
---------------------------------------------------------------------------
ValueError: The name "" is not a valid Python identifier.
}}}
not good because the intent is clear to make precisely x the variable.
So is this breaking incorrect but usable behavior?
{{{
sage: var("x y z")
(x, y, , z)
sage:
}}}
is similar.
Anyway, I withhold judgment on this. Reviewer patch attached, but 'needs
info' on this. At the least it seems reasonable to open a new ticket to
allow the above behavior - one could easily remove empty strings from the
list `names_list` and then complain if there are none left, for instance.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7496#comment:7>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
