Opened 5 years ago

Last modified 5 weeks ago

#23657 needs_work enhancement

Improve Expression member function documentation

Reported by: rws Owned by:
Priority: major Milestone:
Component: documentation Keywords:
Cc: Merged in:
Authors: Ralf Stephan Reviewers:
Report Upstream: N/A Work issues:
Branch: u/rws/23657-1 (Commits, GitHub, GitLab) Commit: 3ba45db080554586c0c4eeb8e91a0968a9025ee3
Dependencies: #23670 Stopgaps:

Status badges

Description

This 1. moves the current expression.pyx document head to relation.pyx and 2. adds a structured index to expression.pyx member functions.

Change History (9)

comment:1 Changed 5 years ago by rws

Branch: → u/rws/23657

comment:2 Changed 5 years ago by rws

Branch: u/rws/23657 → u/rws/23657-1

comment:3 Changed 5 years ago by rws

Authors: → Ralf Stephan
Commit: → 10397499b65791e6064371d2b44733d28e59117c
Status: new → needs_review

New commits:

​103974923657: Improve Expression member function documentation

comment:4 Changed 5 years ago by git

Commit: 10397499b65791e6064371d2b44733d28e59117c → 3ba45db080554586c0c4eeb8e91a0968a9025ee3

Branch pushed to git repo; I updated commit sha1. New commits:

​3ba45db23657: fixes

comment:5 Changed 5 years ago by tscrim

I really do not like (manual) tables of methods (which are not functions in the strict sense, i.e., you should use :meth: instead of :func:). However, I am willing to set a positive review if you use the automated table as done for graph.py.

comment:6 Changed 5 years ago by rws

Status: needs_review → needs_work

comment:7 Changed 5 years ago by rws

Great work in graph.py. I might sort the methods in a different ticket. Having an index at all is of immediate importance.

comment:8 Changed 5 years ago by rws

Dependencies: → #23670

However, we would depend on #23670 because expression.pyx will give the error described there.

comment:9 Changed 5 weeks ago by mkoeppe

Milestone: sage-8.1
Note: See TracTickets for help on using tickets.