Opened 2 years ago

Closed 22 months ago

#26495 closed enhancement (fixed)

Py3 : Fix doctests in module tests/french_book

Reported by: vklein Owned by:
Priority: major Milestone: sage-8.7
Component: python3 Keywords:
Cc: zimmerma, tscrim Merged in:
Authors: Vincent Klein, Frédéric Chapoton Reviewers: Vincent Klein
Report Upstream: N/A Work issues:
Branch: 2a6aae8 (Commits) Commit: 2a6aae804b54637c9584a8e6bf75ddc01fad86a3
Dependencies: Stopgaps:

Description (last modified by chapoton)

For frenchbook doctests errors, I chose to use doctest flag # py2 instead of rewriting the examples.

As the tests are supposed to be the examples of the book i think it's more consistent to just skip them in python 3 rather than rewrite them and have examples which differ form the book.

The remaining doctests errors does not depends on this "module".

Change History (18)

comment:1 Changed 2 years ago by vklein

  • Branch set to u/vklein/26495

comment:2 Changed 2 years ago by vklein

  • Commit set to 57bdc26bc290ac61736a0bfb73a7058f8dcdabe5
  • Status changed from new to needs_review

New commits:

​57bdc26Trac #26495: Py3 use doctest flag `#py 2` for incompati ...

comment:3 Changed 2 years ago by chapoton

  • Cc zimmerma added

comment:4 Changed 2 years ago by zimmerma

thank you for adding me in cc

Note however that tests/french_book is currently being updated with #23572, thus it would be better to apply this patch after #23572 (for which reviewers are welcome).

comment:5 Changed 2 years ago by vklein

  • Dependencies set to #23572
  • Status changed from needs_review to needs_info

comment:6 Changed 2 years ago by vklein

  • Status changed from needs_info to needs_work

I will update this ticket after #23572's merge.

comment:7 Changed 2 years ago by chapoton

  • Milestone changed from sage-8.4 to sage-8.5

comment:8 Changed 2 years ago by chapoton

  • Milestone changed from sage-8.5 to sage-8.7

comment:9 Changed 22 months ago by chapoton

the new-french-book doctests have many errors ; some may be fixed by #26792

comment:10 Changed 22 months ago by chapoton

  • Branch changed from u/vklein/26495 to public/ticket/26495
  • Commit changed from 57bdc26bc290ac61736a0bfb73a7058f8dcdabe5 to ddef784e73dceb70b607cc71aa8d16c033d3e19f
  • Dependencies #23572 deleted

tentative merge

New commits:

​ddef784Merge branch 'u/vklein/26495' in 8.7.b4

comment:11 Changed 22 months ago by git

  • Commit changed from ddef784e73dceb70b607cc71aa8d16c033d3e19f to 5db080fcc808b8926515569a36494c0138407a60

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

​5db080fpy3: some fixes in the book

comment:12 Changed 22 months ago by git

  • Commit changed from 5db080fcc808b8926515569a36494c0138407a60 to a1e8b4fc2bca5dc674908b1794991e52f52068d0

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

​a1e8b4fpy3: more fixes in book doctests

comment:13 Changed 22 months ago by git

  • Commit changed from a1e8b4fc2bca5dc674908b1794991e52f52068d0 to 2a6aae804b54637c9584a8e6bf75ddc01fad86a3

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

​2a6aae8trac 26495 fix doctest

comment:14 Changed 22 months ago by chapoton

  • Status changed from needs_work to needs_review

comment:15 Changed 22 months ago by chapoton

  • Cc tscrim added
  • Description modified (diff)

I suggest that we should positive-review this one and keep other failures for another ticket. See also #26792

comment:16 Changed 22 months ago by vklein

  • Authors changed from Vincent Klein to Vincent Klein, Frédéric Chapoton
  • Reviewers set to Vincent Klein

Ok. Looks good to me. I give positive review to the last three commits. Frederic you set this ticket in positive review if you validate commit 57bdc26

comment:17 Changed 22 months ago by chapoton

  • Status changed from needs_review to positive_review

ok, merci

comment:18 Changed 22 months ago by vbraun

  • Branch changed from public/ticket/26495 to 2a6aae804b54637c9584a8e6bf75ddc01fad86a3
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.