Opened 20 months ago

Closed 18 months ago

#31692 closed enhancement (fixed)

Refactor display behavior of mixed forms

Reported by: Michael Jung Owned by:
Priority: major Milestone: sage-9.4
Component: manifolds Keywords:
Cc: Eric Gourgoulhon, Travis Scrimshaw Merged in:
Authors: Michael Jung Reviewers: Eric Gourgoulhon
Report Upstream: N/A Work issues:
Branch: 6c714ef (Commits, GitHub, GitLab) Commit: 6c714efcfdcd9d9ce7755574eab8c7202ad08c25
Dependencies: Stopgaps:

Status badges

Description

We refactor the display behavior of mixed forms, in particular:

  • We remove square brackets around homogeneous components since it might lead to confusion with cohomology elements (or any other equivalence classes).
  • We remove zero-terms to make the output cleaner.

Change History (9)

comment:1 Changed 20 months ago by Michael Jung

Branch: → u/gh-mjungmath/refactor_display_behavior_of_mixed_forms

comment:2 Changed 20 months ago by Michael Jung

Commit: → 8a0a9b65a2380ddee5a8720c12bf5b67f171d6fb
Status: new → needs_review

New commits:

​89373d4Trac #31692: refactor display_expansion
​8a0a9b6Trac #31692: fix doctests

comment:3 Changed 20 months ago by Michael Jung

Authors: → Michael Jung

I always forget the authorship...

comment:4 Changed 20 months ago by Eric Gourgoulhon

Status: needs_review → needs_work

The patchbot says 

File "src/sage/manifolds/differentiable/vector_bundle.py", line 242, in sage.manifolds.differentiable.vector_bundle.DifferentiableVectorBundle.characteristic_class
Failed example:
    p_form = p.get_form(nab); p_form.display_expansion()
Expected:
    p(TM, nabla_g) = [1] + [0] + [0] + [0] + [0]
Got:
    p(TM, nabla_g) = 1

comment:5 Changed 20 months ago by Michael Jung

Is there a way to fixdoctesting all files in a folder at once, optimally with multicore support?

comment:6 Changed 20 months ago by git

Commit: 8a0a9b65a2380ddee5a8720c12bf5b67f171d6fb → 6c714efcfdcd9d9ce7755574eab8c7202ad08c25

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

​6c714efTrac #31692: doctest fix

comment:7 Changed 20 months ago by Michael Jung

Status: needs_work → needs_review

There we go.

comment:8 Changed 20 months ago by Eric Gourgoulhon

Reviewers: → Eric Gourgoulhon
Status: needs_review → positive_review

LGTM.

comment:9 Changed 18 months ago by Volker Braun

Branch: u/gh-mjungmath/refactor_display_behavior_of_mixed_forms → 6c714efcfdcd9d9ce7755574eab8c7202ad08c25
Resolution: → fixed
Status: positive_review → closed
Note: See TracTickets for help on using tickets.