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

Get rid of sage.combinat.cartesian_product #19192

Closed
videlec opened this issue Sep 12, 2015 · 5 comments
Closed

Get rid of sage.combinat.cartesian_product #19192

videlec opened this issue Sep 12, 2015 · 5 comments

Comments

@videlec
Copy link
Contributor

videlec commented Sep 12, 2015

We just make CartesianProduct an alias for sage.sets.cartesian_product.CartesianProduct (with some tweak to support the previous behavior which considered the cartesian product as a set of lists).

sage.combinat.cartesian_product.CartesianProduct is deprecated.

CC: @nthiery @nathanncohen

Component: combinatorics

Reviewer: Travis Scrimshaw

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

@videlec videlec added this to the sage-6.9 milestone Sep 12, 2015
@videlec

This comment has been minimized.

@videlec
Copy link
Contributor Author

videlec commented Sep 13, 2015

Changed dependencies from #18411 to none

@videlec
Copy link
Contributor Author

videlec commented Sep 13, 2015

comment:2

superseded by #18411

@videlec
Copy link
Contributor Author

videlec commented Sep 13, 2015

Changed author from Vincent Delecroix to none

@videlec videlec removed this from the sage-6.9 milestone Sep 13, 2015
@tscrim
Copy link
Collaborator

tscrim commented Sep 13, 2015

Reviewer: Travis Scrimshaw

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants