Opened 18 months ago

Last modified 8 months ago

#27150 new enhancement

Better UI for Python 3 Sage doctesting

Reported by: vdelecroix Owned by:
Priority: minor Milestone:
Component: doctest framework Keywords:
Cc: chapoton Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by jdemeyer)

running sage -t in a Python 2 or Python 3 installation of Sage results in a seemingly wrong set of --optional flags

$ sage -t XXX
...
Using --optional=dochtml,memlimit,mpir,python2,sage
...

In a Python 2 installation the py2 should be there (and the python2 appearing because of a is quite weird). While on Python 3 it should be py3.

See also this sage-devel thread.

Change History (10)

comment:1 Changed 18 months ago by vdelecroix

  • Description modified (diff)

comment:2 Changed 18 months ago by jdemeyer

  • Description modified (diff)
  • Priority changed from critical to minor
  • Summary changed from Fix Python 3 Sage doctesting to Better UI for Python 3 Sage doctesting

It's just a UI issue. Everything is behaving as it should be...

comment:3 Changed 18 months ago by vdelecroix

I don't agree: python2 doctesting is also broken. The py2 tag is not in the optional list and hence ignored.

comment:4 Changed 18 months ago by jdemeyer

Are you really sure?

comment:5 Changed 18 months ago by vdelecroix

It depends what you mean by "normal". More precisely:

  • the optional py2/py3 do not appear on the command line due to the filtering in line 1018 of doctest/control.py
    return ",".join(sorted(tags - auto_optional_tags))
    
  • the python2 appears in the list because python2 is an optional package (is it installed under any circumstances?)

Both look quite artificial to me.

Conclusion: doctests are not broken - behavior is very confusing.

comment:6 Changed 18 months ago by jdemeyer

  • Type changed from defect to enhancement

Everything is working as designed but the design might be confusing. That's why I said that it's an UI issue.

comment:7 Changed 18 months ago by vdelecroix

@jeroen: Do you know whether python2 is always installed? (it got installed on my fresh py3 build). Why is it optional?

comment:8 Changed 16 months ago by embray

  • Milestone changed from sage-8.7 to sage-8.8

Ticket retargeted after milestone closed (if you don't believe this ticket is appropriate for the Sage 8.8 release please retarget manually)

comment:9 Changed 13 months ago by embray

  • Milestone sage-8.8 deleted

As the Sage-8.8 release milestone is pending, we should delete the sage-8.8 milestone for tickets that are not actively being worked on or that still require significant work to move forward. If you feel that this ticket should be included in the next Sage release at the soonest please set its milestone to the next release milestone (sage-8.9).

comment:10 Changed 8 months ago by chapoton

  • Cc chapoton added; fchapoton removed
Note: See TracTickets for help on using tickets.