#17351: Sage Learn_sat AttributeError
-------------------------------------+-------------------------------------
Reporter: ounidjoo | Owner:
Type: defect | Status: needs_info
Priority: major | Milestone: sage-6.5
Component: packages: | Resolution:
optional | Merged in:
Keywords: | Reviewers:
Authors: Martin Albrecht | Work issues:
Report Upstream: N/A | Commit:
Branch: | 775b1a6471f46c2d71dc8a722f0e7730f2c62fd7
u/malb/t17351_learn_sat | Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by malb):
Replying to [comment:9 vdelecroix]:
> Replying to [comment:8 malb]:
> > Replying to [comment:7 vdelecroix]:
> > > Fine. Please update the description of the ticket...
> >
> > Sorry, but I don't see why. Somebody posted a bug. I fixed it and
provide a doctest that it is fixed.
>
> The ticket description relates a bug but does not describe the changes
you provide. Moreover this description will appear in the git history of
Sage and there is no place for "hello" or "could you help me".
>
> > > Why `Sequence` is still used in some place?
> >
> > `Sequence` in general is fine. In this particular instance it might
get passed an empty set when it cannot determine that it really should
construct a `PolynomialSequence`.
>
> From my naive point of view, it would be less `import` at the beginning
of the file if we replace all `Sequence` by `PolynomialSequence`. Would it
be worse that way?
Thanks for explaining. I did both.
--
Ticket URL: <http://trac.sagemath.org/ticket/17351#comment:14>
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.
