Opened 3 years ago

Closed 3 years ago

#28769 closed defect (fixed)

Scalar field raise AttributeError on show_identifiers call

Reported by: Léo Brunswic Owned by:
Priority: major Milestone: sage-9.0
Component: geometry Keywords: Manifold, save_
Cc: Eric Gourgoulhon Merged in:
Authors: Eric Gourgoulhon Reviewers: Léo Brunswic
Report Upstream: N/A Work issues:
Branch: a1bee7a (Commits, GitHub, GitLab) Commit: a1bee7a0e89378dc8769e57e966b26681235c830
Dependencies: Stopgaps:

Status badges

Description

The following snippet raises AttributeError 

M = Manifold(1, 'M')
C.<x> = M.chart()
phi = M.scalar_field(0)
show_identifiers()

This prevents saving any sage session containing a scalar field object.

Change History (6)

comment:1 Changed 3 years ago by Eric Gourgoulhon

Authors: Léo Brunswic → Eric Gourgoulhon
Branch: → public/manifolds/scalarfield__eq__-28769
Commit: → a1bee7a0e89378dc8769e57e966b26681235c830

New commits:

​a1bee7aMake ScalarField.__eq__() more robust

comment:2 Changed 3 years ago by Eric Gourgoulhon

Status: new → needs_review

The issue was due to an exception not catched in ScalarField.__eq__(). This is fixed in the above commit.

comment:3 Changed 3 years ago by Léo Brunswic

Status: needs_review → positive_review

comment:4 Changed 3 years ago by Eric Gourgoulhon

Thanks for reporting the bug and for the review!

comment:5 Changed 3 years ago by Eric Gourgoulhon

Reviewers: → Léo Brunswic

comment:6 Changed 3 years ago by Volker Braun

Branch: public/manifolds/scalarfield__eq__-28769 → a1bee7a0e89378dc8769e57e966b26681235c830
Resolution: → fixed
Status: positive_review → closed
Note: See TracTickets for help on using tickets.