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.

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

Add the downward monotonic cone to the cone catalogue #30200

Closed
orlitzky opened this issue Jul 22, 2020 · 22 comments · Fixed by #37802
Closed

Add the downward monotonic cone to the cone catalogue #30200

orlitzky opened this issue Jul 22, 2020 · 22 comments · Fixed by #37802

Comments

@orlitzky
Copy link
Contributor

Depends on #30128
Depends on #30153
Depends on #32877

Component: geometry

Author: Michael Orlitzky

Branch/Commit: u/mjo/ticket/30200 @ 7268365

Issue created by migration from https://trac.sagemath.org/ticket/30200

@orlitzky orlitzky added this to the sage-9.2 milestone Jul 22, 2020
@orlitzky
Copy link
Contributor Author

Branch: u/mjo/ticket/30200

@orlitzky
Copy link
Contributor Author

Last 10 new commits:

554282aTrac #30128: enforce sourcing of sage-env-config before src/bin/sage-env.
7a91c05build/make/Makefile.in: Add target SPKG-no-deps for all SPKGs
4fc909csrc/bin/sage: Repair ./sage -b etc. by using make sagelib-clean, make sagelib-no-deps
03a2dbebuild/make/Makefile.in: Make SPKG-no-deps targets phony; for script packages, create the correct installation stamp file
62ddd02build/make/Makefile.in [SPKG-no-deps]: Do not forget to source sage-env-config
830843btrac 30153: add, clarify some comments
1a0e5c3Trac #30153: factor out $(1)-no-deps targets in build/make/Makefile.in.
7456bf5Trac #30200: add the global Niez1998 reference.
8a9a83aTrac #30200: add the downward-monotonic cone to the cone catalogue.
e137879Trac #30200: fix the implementation of the "majorization order."

@orlitzky
Copy link
Contributor Author

Commit: e137879

@orlitzky
Copy link
Contributor Author

Author: Michael Orlitzky

@orlitzky
Copy link
Contributor Author

Dependencies: #30128, #30153

@mkoeppe
Copy link
Contributor

mkoeppe commented Jul 25, 2020

comment:2

This looks clean but I am not sure how widespread the name "downward monotonic cone" really is. Google finds only 10 hits of this phrase (and 0 hits for "downward monotone cone"). In fact, looking at the ticket I somehow expected to see an implementation of the nonpositive orthant...

So how about calling it cones.nonincreasing_sequences instead.

@orlitzky
Copy link
Contributor Author

comment:3

Replying to @mkoeppe:

This looks clean but I am not sure how widespread the name "downward monotonic cone" really is. Google finds only 10 hits of this phrase (and 0 hits for "downward monotone cone"). In fact, looking at the ticket I somehow expected to see an implementation of the nonpositive orthant...

So how about calling it cones.nonincreasing_sequences instead.

I tortured myself about this name for hours in order to include a split-second mention of it in a recent paper. The first application of it is to isotone or isotonic regression, where of course "iso" only means "same" and doesn't specify the direction. More recently and in other contexts, "monotone" has become more popular but suffers the same problem. You can find the ambiguous "monotone cone" in recent work on cone projections by Németh et al., for example.

In any case, I'm forced to add an adjective to either of those, which have the only claim to being "official." I've chosen "downward monotonic" because it's the only option that

  • has been used to mean what I'm using it to mean
  • has not been used to mean anything else (as far as I know)

So even though there aren't that many mentions of it, the name "downward monotonic" at least appears in a few published papers, and then always to mean the cone that I've implemented. The "cone of nonincreasing sequences" on the other hand, has been used instead to refer to the same cone in a little-ell space. "Isotonic" has likewise been used much more generally.

Finally, at least Gourion and Seeger have defined the "upward monotonic cone" officially, so if in the future we need to add its counterpart we have a reference for that.

@mkoeppe
Copy link
Contributor

mkoeppe commented Jul 28, 2020

comment:4

Replying to @orlitzky:

The "cone of nonincreasing sequences" on the other hand, has been used instead to refer to the same cone in a little-ell space.

Well, the factory could be generalized to cones.nonincreasing_sequences(oo) to cover that, if/when you add support for non-polyhedral cones.

@orlitzky
Copy link
Contributor Author

comment:5

"nonincreasing sequences" is indeed a better name, but I'd rather not come up with Yet Another Name for the thing unless it's a spectacularly better name, or the name change is forced on us by some (e.g.) python restriction. No one has ever called it the cone of nonincreasing sequences before, so it's hard for me to justify making up a new name whose only google hits will be in the sage documentation. (Note that "downward monotonic" already appears in the docs for the Schur cone.)

@mkoeppe
Copy link
Contributor

mkoeppe commented Jul 29, 2020

comment:6

Replying to @orlitzky:

No one has ever called it the cone of nonincreasing sequences before

This is unsubstantiated.

Note that it is not a "new name" of a cone. It is a description of its elements -- which is immediately accessible to anyone who has taken a calculus class.

@orlitzky
Copy link
Contributor Author

comment:7

Replying to @mkoeppe:

Replying to @orlitzky:

No one has ever called it the cone of nonincreasing sequences before

This is unsubstantiated.

Indeed, Niezgoda also calls it the cone of nonincreasing sequences once in "Bifractional inequalities and convex cones," but that's the only paper I was able to find using that name.

Note that it is not a "new name" of a cone. It is a description of its elements -- which is immediately accessible to anyone who has taken a calculus class.

To match the other adjective-form names I would prefer "nonincreasing sequential," but I still think "downward monotonic" is a more evocative name. If we ever wind up with both, having "nonincreasing sequential" and "nondecreasing sequential" where the "increasing" one is decreasing and the "decreasing" one is increasing is more confusing than it has to be. The notation for the downward monotonic cone also justifies its name; the symbol x^{\downarrow} is standard to indicate the vector whose components are the components of x but arranged in nonincreasing order. The downward-monotonic cone is thus (R<sup>{n})</sup>{\downarrow}, and the upward-monotonic cone is (R<sup>{n})</sup>{\uparrow}. Since the downward cone is just R^n with a "down" after it and likewise for the upward cone, they're easy to remember.

In any case, I'm not prepared to argue about this forever. I didn't invent the cone, and isotonic/monotonic are the common names for it, with "monotonic" going back at least to Hardy, Littlewood, and Polya. I see no good reason to rename it (beyond the necessary mention of the direction of monotonicity) in sage.

@mkoeppe
Copy link
Contributor

mkoeppe commented Jul 30, 2020

comment:8

The cone of nonincreasing sequences is an "everyday" mathematical object. If we add it to Sage, it should be named in a way that is as universally understood -- not using jargon from a narrow list of papers.

By the way, also "monotonic cone" only has 21 Google hits (I know, I know, Polya was more a fan of Bing than of Google search), and "monotone cone" only 70.

@orlitzky
Copy link
Contributor Author

comment:9

This doesn't "need work" any more now than it would if I changed the name. Your own search results show that I've chosen the more popular name of the two, and this for inclusion in a catalogue of cones whose sole purpose is to make them easily accessible by name. Please just let someone else review it.

@mkoeppe
Copy link
Contributor

mkoeppe commented Jul 30, 2020

comment:10

Replying to @orlitzky:

I've chosen the more popular name of the two

of what two?

@orlitzky
Copy link
Contributor Author

comment:11

downward monotonic cone and cone of nonincreasing sequences

@mkoeppe mkoeppe modified the milestones: sage-9.2, sage-9.3 Oct 24, 2020
@mkoeppe
Copy link
Contributor

mkoeppe commented Mar 15, 2021

comment:13

Setting new milestone based on a cursory review of ticket status, priority, and last modification date.

@mkoeppe mkoeppe modified the milestones: sage-9.3, sage-9.4 Mar 15, 2021
@mkoeppe
Copy link
Contributor

mkoeppe commented Jul 19, 2021

comment:14

Setting a new milestone for this ticket based on a cursory review.

@mkoeppe mkoeppe modified the milestones: sage-9.4, sage-9.5 Jul 19, 2021
@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 1, 2021

Changed commit from e137879 to 931ca5f

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 1, 2021

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

f976e17Trac #32877: remove set_random_seed() calls from doctests.
840b753Trac #30200: add the global Niez1998 reference.
5b4f844Trac #30200: add the downward-monotonic cone to the cone catalogue.
931ca5fTrac #30200: fix the implementation of the "majorization order."

@orlitzky
Copy link
Contributor Author

orlitzky commented Dec 2, 2021

Changed dependencies from #30128, #30153 to #30128, #30153, #32877

@mkoeppe mkoeppe modified the milestones: sage-9.5, sage-9.6 Dec 27, 2021
@mkoeppe mkoeppe modified the milestones: sage-9.6, sage-9.7 May 3, 2022
@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 13, 2022

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

63fa368Trac #30200: add the global Niez1998 reference.
1805a14Trac #30200: add the downward-monotonic cone to the cone catalogue.
7268365Trac #30200: fix the implementation of the "majorization order."

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 13, 2022

Changed commit from 931ca5f to 7268365

@mkoeppe mkoeppe modified the milestones: sage-9.7, sage-9.8 Aug 31, 2022
@mkoeppe mkoeppe removed this from the sage-9.8 milestone Jan 29, 2023
vbraun pushed a commit to vbraun/sage that referenced this issue Apr 28, 2024
    
This an an old PR that I never migrated from trac. We have a catalogue
of convex cones accessible through `cones.foo()`, and this PR adds the
downward monotonic cone to the catalogue. It will also close
sagemath#30200.
    
URL: sagemath#37802
Reported by: Michael Orlitzky
Reviewer(s): Matthias Köppe
vbraun pushed a commit to vbraun/sage that referenced this issue May 2, 2024
    
This an an old PR that I never migrated from trac. We have a catalogue
of convex cones accessible through `cones.foo()`, and this PR adds the
downward monotonic cone to the catalogue. It will also close
sagemath#30200.
    
URL: sagemath#37802
Reported by: Michael Orlitzky
Reviewer(s): Matthias Köppe
@mkoeppe mkoeppe added this to the sage-10.4 milestone May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants