Changes between Initial Version and Version 3 of Ticket #28422


Ignore:
Timestamp:
08/29/19 15:48:48 (2 years ago)
Author:
egourgoulhon
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #28422

    • Property Status changed from new to needs_review
    • Property Cc SimonKing tscrim added
    • Property Summary changed from Change keyword argument 'verbose' to 'check' in CoordChange.set_inverse(() to Change keyword argument 'verbose' to 'check' in CoordChange.set_inverse() and set the default to True
    • Property Branch changed from to public/manifolds/check_set_inverse-trac28422
    • Property Commit changed from to 26a47e058dfb941c378c3e0e17f1684611b57691
  • Ticket #28422 – Description

    initial v3  
    1 As pointed out in this [https://groups.google.com/d/msg/sage-devel/AilDfydrnSI/XhO1EbALEQAJ sage-devel post], the keyword `verbose=True` used to trigger the check of the inverse of a transition map in `CoordChange.set_inverse()` would be better named `check=True`. This performed in this ticket. Moreover, `check=True` is now the default (previously, it was `verbose=False`) and the printed output of the check is improved. 
     1As pointed out in this [https://groups.google.com/d/msg/sage-devel/AilDfydrnSI/XhO1EbALEQAJ sage-devel post], the keyword `verbose=True` used to trigger the check of the inverse of a transition map in `CoordChange.set_inverse()` would be better named `check=True`. This is performed in this ticket. Moreover, `check=True` is now the default (previously, it was `verbose=False`) and the printed output of the check has been improved.