Opened 8 years ago
Closed 8 years ago
#14473 closed defect (fixed)
Old deprecation warnings
Reported by: | ncohen | Owned by: | sage-combinat |
---|---|---|---|
Priority: | major | Milestone: | sage-5.10 |
Component: | combinatorics | Keywords: | |
Cc: | Merged in: | sage-5.10.beta4 | |
Authors: | Nathann Cohen | Reviewers: | Travis Scrimshaw |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
This ticket removes some old deprecation warnings.
Nathann
Attachments (1)
Change History (13)
comment:1 Changed 8 years ago by
- Status changed from new to needs_review
comment:2 follow-up: ↓ 3 Changed 8 years ago by
- Reviewers set to Travis Scrimshaw
- Status changed from needs_review to needs_work
comment:3 in reply to: ↑ 2 Changed 8 years ago by
Hellooooooooo !!
- Ticket #14464 removes the deprecated copy mehtods from
yang_baxter_graph.py
.
Oh. Right !
- I would not remove the
element_class
method untilCombintorialClass
has been fully removed (I've been working on getting this done, but there are some entanglements I've been trying to work out. Also, you'll notice the function is not actually deprecated).
Ahem. Just being stupid v_v
I'll continue the review once these are changed.
Done ! And thank you for doing the dirty job of dealing with CombinatorialClass
:-/
Nathann
comment:4 Changed 8 years ago by
- Status changed from needs_work to needs_review
comment:5 Changed 8 years ago by
Ping ? :-)
Nathann
comment:6 follow-up: ↓ 7 Changed 8 years ago by
Hey Nathann,
I've talked it over with Nicolas and I'm still working on a draft of the restructuring of the combinat doc folder. There are some tests with doc-linking that I'm having to do as well to get it to a state we want. I also have gotten distracted by some mathematics and some other patches. This will absolutely be done by the Sage Days in Orsay.
Best,
Travis
comment:7 in reply to: ↑ 6 Changed 8 years ago by
Hellooooooo Travis !
I've talked it over with Nicolas and I'm still working on a draft of the restructuring of the combinat doc folder. There are some tests with doc-linking that I'm having to do as well to get it to a state we want. I also have gotten distracted by some mathematics and some other patches. This will absolutely be done by the Sage Days in Orsay.
? Well, it is all very nice but how is that related with this patch ? It's small, it removes code and changes the doc's depth O_o
Nathann
comment:8 follow-up: ↓ 9 Changed 8 years ago by
Hey Nathann,
I do not like the result of decreasing the combinatorics doc's depth. The principle is to improve the readability of the combinatorics documentation, and that is what the review patch aims to do. Plus it was something I was already working on. If you want to remove the old deprecations, we can do that now, but I do not want the doc's depth to simply decrease. I believe this makes it more difficult to look through the reference manual (for things not sitting at the combinatorics level, ex. symmetric functions, crystals).
Best,
Travis
Changed 8 years ago by
comment:9 in reply to: ↑ 8 Changed 8 years ago by
Yo !
I do not like the result of decreasing the combinatorics doc's depth.
Oh. Ok.
It really is a mess right now, you know..
http://www.sagemath.org/doc/reference/combinat/index.html
The principle is to improve the readability of the combinatorics documentation, and that is what the review patch aims to do.
Which review patch are you talking about ?
Plus it was something I was already working on.
Well. Yeah, but it's not done yet...
If you want to remove the old deprecations, we can do that now
Cool, let's do it then !
, but I do not want the doc's depth to simply decrease. I believe this makes it more difficult to look through the reference manual (for things not sitting at the combinatorics level, ex. symmetric functions, crystals).
Well, symmetric functions and crystal would both have an entry in the reduced listing. And as there would be fewer entries one would have an actually chance to see them.
The other point is that reducing the depth would have made people take care of the doc instead of putting anything there randomly to be used with "Ctrl + f".
But if you work on this right now and if you say that the patch will be ready soon, after all.. It has been like that for so long !
I just updated the patch, which now does not change the depth of the combinat doc page. It still changes the depth of the posets page, but you can check there http://www.sagemath.org/doc/reference/combinat/posets.html that only "Classes and functions" would be removed.
Nathann
comment:10 follow-up: ↓ 11 Changed 8 years ago by
- Description modified (diff)
- Status changed from needs_review to positive_review
- Summary changed from Old deprecation warnings, and depth of doc index to Old deprecation warnings
Hey Nathann,
Since the re-organizing is not that close to being done (at least by what I believe your definition of close is), I'm setting this to positive review. I will create a new ticket for the reorganization of the combinatorics doc and get that posted to trac as soon as it's done.
Thanks,
Travis
comment:11 in reply to: ↑ 10 Changed 8 years ago by
Hellooooooooooooooooooooooo !!!
Since the re-organizing is not that close to being done (at least by what I believe your definition of close is),
Ahahaha :-)
I'm setting this to positive review. I will create a new ticket for the reorganization of the combinatorics doc and get that posted to trac as soon as it's done.
Thaaaaaaaaaaanks ! I will look at your other patch too, I also want this combinat doc index to become readable eventually :-P
Nathann
comment:12 Changed 8 years ago by
- Merged in set to sage-5.10.beta4
- Resolution set to fixed
- Status changed from positive_review to closed
Hey Nathann,
Two things first off:
yang_baxter_graph.py
.element_class
method untilCombintorialClass
has been fully removed (I've been working on getting this done, but there are some entanglements I've been trying to work out. Also, you'll notice the function is not actually deprecated).I'll continue the review once these are changed.
Thanks,
Travis