#16757: Organize the V(m,t) vectors into a dictionary
-------------------------------------+-------------------------------------
Reporter: ncohen | Owner:
Type: enhancement | Status: positive_review
Priority: major | Milestone: sage-6.4
Component: combinatorial | Resolution:
designs | Merged in:
Keywords: | Reviewers: Vincent Delecroix
Authors: Nathann Cohen | Work issues:
Report Upstream: N/A | Commit:
Branch: | 57c00f0a5d0ed574d33c0e5c0b81e89958753661
u/vdelecroix/16757 | Stopgaps:
Dependencies: #16722 |
-------------------------------------+-------------------------------------
Changes (by ncohen):
* status: needs_review => positive_review
* commit: 9a57f1312acf35c29e085a20f46a18da698c5c06 =>
57c00f0a5d0ed574d33c0e5c0b81e89958753661
* branch: u/ncohen/16757 => u/vdelecroix/16757
Comment:
Yoooooooooo !!
> 1) The documentation in the dictionary is a bit strange... but it is
modified in #16763. So it is fine for me as I will review the latter in a
minute.
Yep, sorry for that ! I got an email from Julian about those references
after submitting that commit.
> 2) There is no need for an empty function to have a doctest. In
`u/vdelecroix/16757` I removed it and use a string alone. Drawback: there
will be some conflict with #16763.
No prob, shouldn't be hard to fix. Thanks again !
Nathann
----
New commits:
||[http://git.sagemath.org/sage.git/commit/?id=57c00f0a5d0ed574d33c0e5c0b81e89958753661
57c00f0]||{{{trac #16757: doctest simplication}}}||
--
Ticket URL: <http://trac.sagemath.org/ticket/16757#comment:6>
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.
