Opened 4 years ago

Last modified 23 months ago

#25096 needs_work enhancement

Simplify tachyon interface

Reported by: saraedum Owned by:
Priority: trivial Milestone: sage-8.3
Component: interfaces Keywords: debian
Cc: infinity0 Merged in:
Authors: Julian Rüth Reviewers:
Report Upstream: N/A Work issues:
Branch: u/saraedum/25096 (Commits, GitHub, GitLab) Commit: 1c63a792d48a89d1d53b855fd173804da67099cf
Dependencies: Stopgaps:

Status badges

Description

by dropping usage().

I don't think it's necessary to expose this in the Sage interface. Its doctest fails for Debian at least (where tachyon's usage screen has been patched?) If people are in IPython or a Jupyter notebook, they can just type !tachyon to see the same information. In theory this breaks the existing interface but I am quite sure nobody is relying on this.

Change History (12)

comment:1 Changed 4 years ago by saraedum

  • Branch set to u/saraedum/simplify-tachyon

comment:2 Changed 4 years ago by saraedum

  • Commit set to 082fdaabb3677ac803d74cd5b0c5d6f7be108b3f
  • Status changed from new to needs_review

New commits:

082fdaaDrop tachyon's usage()

comment:3 Changed 4 years ago by saraedum

I think we can live without the pager feature. The output is only 82 lines. That's not too much for people to scroll I think.

comment:4 Changed 4 years ago by saraedum

This upstreams part of Debian's u2-fix-trivial-test-cases.patch btw.

comment:5 Changed 3 years ago by saraedum

  • Milestone changed from sage-8.2 to sage-8.3

comment:6 Changed 3 years ago by saraedum

  • Branch u/saraedum/simplify-tachyon deleted
  • Commit 082fdaabb3677ac803d74cd5b0c5d6f7be108b3f deleted

comment:7 Changed 3 years ago by saraedum

  • Branch set to u/saraedum/25096

comment:8 Changed 3 years ago by git

  • Commit set to ddc714b5e31bf1fc963c4f9bed699d7f1b9fe3d2

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

ddc714bMerge 25096 and develop

comment:9 Changed 3 years ago by saraedum

  • Keywords debian added

New commits:

082fdaaDrop tachyon's usage()
fcb2808Merge trac/develop and 25096
ddc714bMerge 25096 and develop

comment:10 Changed 3 years ago by git

  • Commit changed from ddc714b5e31bf1fc963c4f9bed699d7f1b9fe3d2 to 1c63a792d48a89d1d53b855fd173804da67099cf

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

1c63a79fix pyflakes warnings

comment:11 Changed 3 years ago by saraedum

The patchbot fails due to a problem in the cygwin build that is unrelated to this ticket's changes. This still needs review.

comment:12 Changed 23 months ago by chapoton

  • Status changed from needs_review to needs_work

red branch

Note: See TracTickets for help on using tickets.