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

More crystals speedups #14686

Open
tscrim opened this issue Jun 4, 2013 · 4 comments
Open

More crystals speedups #14686

tscrim opened this issue Jun 4, 2013 · 4 comments
Assignees
@sagetrac-sage-combinat
Labels
c: combinatorics p: major / 3 s: needs info t: enhancement

Comments

@tscrim
Copy link
Collaborator

tscrim commented Jun 4, 2013

Followup to #14516 which does the following:

  • Instead of e(), f(), epsilon(), phi() checking the index set each time, these functions will check to make sure i is in the index set, then call _e(), _f(), _epsilon(), _phi() which assume i is valid input and respectively call the private methods.

Depends on #14516
Depends on #14573
Depends on #14143

CC: @sagetrac-sage-combinat @bsalisbury1 @anneschilling @nthiery

Component: combinatorics

Keywords: crystals speedup optimization

Author: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/14686
@tscrim tscrim added this to the sage-5.11 milestone Jun 4, 2013
@tscrim
Copy link
Collaborator Author

tscrim commented Jun 4, 2013

Changed dependencies from #14516 to #14516 #14573

@tscrim
Copy link
Collaborator Author

tscrim commented Jun 5, 2013

Changed dependencies from #14516 #14573 to #14516 #14573 #14143

@tscrim
Copy link
Collaborator Author

tscrim commented Jun 5, 2013

Attachment: trac_14686-crystals_speedup_functions-ts.patch.gz

@tscrim
Copy link
Collaborator Author

tscrim commented Jul 18, 2013

comment:3

Until we decide on an approach to handle checks on user inputs only.

@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@mkoeppe mkoeppe removed this from the sage-6.4 milestone Dec 29, 2022
@tscrim tscrim mentioned this issue Jan 16, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sagetrac-sage-combinat sagetrac-sage-combinat

Labels
c: combinatorics p: major / 3 s: needs info t: enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@tscrim @jdemeyer @mkoeppe