Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

proper names should start with a capital letter #15894

Closed
fchapoton opened this issue Mar 6, 2014 · 12 comments
Closed

proper names should start with a capital letter #15894

fchapoton opened this issue Mar 6, 2014 · 12 comments
Labels
p: minor / 4 t: enhancement
Milestone
sage-6.2

Comments

@fchapoton
Copy link
Contributor

One should look for instances of coxeter, dynkin, weyl, weil, bruhat, cartan, borel, galois that should be written with a capital letter.

CC: @sagetrac-sage-combinat

Component: misc

Keywords: typography

Author: Frédéric Chapoton

Branch/Commit: 8c9c684

Reviewer: André Apitzsch

Issue created by migration from https://trac.sagemath.org/ticket/15894
@fchapoton fchapoton added this to the sage-6.2 milestone Mar 6, 2014
@fchapoton
Copy link
Contributor Author

New commits:

4f8ada8trac #15xxx proper names should start by a capital letter

@fchapoton
Copy link
Contributor Author

Commit: 4f8ada8

@fchapoton
Copy link
Contributor Author

Branch: u/chapoton/15894

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 6, 2014

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

1075660trac #15894 changin back the category names
80e8727trac #15894 another backward change

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 6, 2014

Changed commit from 4f8ada8 to 80e8727

@fchapoton
Copy link
Contributor Author

comment:4

I have removed the changes affecting the categories, as this was causing failing doctests. The branch should now be ready for review.

@fchapoton

This comment has been minimized.

Sign in to view
@a-andre
Copy link

a-andre commented Mar 9, 2014

comment:5

Looks good, but could you also change the remaining

warning::

to

.. warning::

in src/sage/combinat/root_system/root_lattice_realizations.py.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 9, 2014

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

8c9c684trac #15894 modif of two warnings

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 9, 2014

Changed commit from 80e8727 to 8c9c684

@a-andre
Copy link

a-andre commented Mar 9, 2014

Reviewer: André Apitzsch

@vbraun
Copy link
Member

vbraun commented Mar 11, 2014

Changed branch from u/chapoton/15894 to 8c9c684

@vbraun vbraun closed this as completed in 3592c0a Mar 11, 2014
tscrim pushed a commit to tscrim/sage that referenced this issue Jun 1, 2023
Merge branch 'develop' into public/algebras/fp_algebras
48fcf2f 
* develop: (100 commits)
  Updated Sage version to 6.2.beta4
  added documentation, patchlevel bump
  trac sagemath#15894 modif of two warnings
  delete cruft from the guava package
  trac sagemath#15893 review patch
  trac sagemath#15894 another backward change
  trac sagemath#15894 changin back the category names
  trac #15xxx proper names should start by a capital letter
  Fix changelog entry for ell_wp
  trac sagemath#15893: graphs.petersen_family
  Fixes to code from sagemath#9505.
  Fix the banner displayed by sage -h
  trac sagemath#15466: remove dead import
  trac sagemath#15795 line wrap
  trac sagemath#15795 redone with ( ::)
  Fraction to power series: doctests
  Fraction to power series conversion when the fraction coerces into the base ring
  15698: Allow construction of power series from fractions
  Create *unevaluated* integrals when converting sage integrals to sympy
  Use evaluate=False in SympyConverter too
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
p: minor / 4 t: enhancement
Projects
None yet
Milestone
sage-6.2
Development

No branches or pull requests
3 participants
@vbraun @fchapoton @a-andre