#20103: Some clean up in simplicial complexes
-------------------------------------+-------------------------------------
Reporter: jhpalmieri | Owner:
Type: enhancement | Status: needs_review
Priority: minor | Milestone: sage-7.1
Component: algebraic | Resolution:
topology | Merged in:
Keywords: | Reviewers:
Authors: John Palmieri | Work issues:
Report Upstream: N/A | Commit:
Branch: u/jhpalmieri | 6fb881a4de6cfa94cad0481120cbd2c48d3ee077
/complex-cleanup | Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by tscrim):
Some quick comments:
- The input block bullets are usually not considered to be sentences and
typically lack periods.
- Use `@abstract_method` for the generic `alexander_whitney`.
- I think this would benefit from using `@total_ordering` (in reality, we
probably should implement a generic `__ne__` for `Element`...) so we don't
have so many little methods for all of the comparison operators.
- I'm actually not sure if the output of a (python) `set` is (attempted to
be) sorted as for a `dict`. So I'm not sure if we need to explicitly call
`sorted`.
- Should we cythonize `algebraic_topological_model` now? It probably won't
be too hard and we could get some speedup from that.
--
Ticket URL: <http://trac.sagemath.org/ticket/20103#comment:4>
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 https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
