Wrapper class for Sage sets as SymPy sets

[mkoeppe]

We add a _sympy_ method to all sets. It creates an instance of the new class sage.interfaces.sympy_wrapper.SageSet, which is a subclass of SymPy's Set class.

Part of Meta-ticket #31926: Connect Sage sets to sympy sets

CC: @kcrisman @tscrim

Component: interfaces

Author: Matthias Koeppe

Branch/Commit: c06c965

Reviewer: Travis Scrimshaw

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

[mkoeppe]

Branch: u/mkoeppe/wrapper_class_for_sage_sets_as_sympy_sets

[mkoeppe]

comment:3

Here's a beginning. This obviously needs many more examples/tests

---

New commits:

6e5cac6

sage.interfaces.sympy_wrapper, Sets.ParentMethods._sympy_: New

[mkoeppe]

Author: Matthias Koeppe

[mkoeppe]

Commit: 6e5cac6

[sagetrac-git]

Changed commit from 6e5cac6 to 3cac256

[sagetrac-git]

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

3cac256

sage.interfaces.sympy_wrapper: Add doctests

[sagetrac-git]

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

eef604e

SageSet: Finish docstrings; handle symbolic _contains

[sagetrac-git]

Changed commit from 3cac256 to eef604e

[sagetrac-git]

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

2baae58

Sets.ParentMethods._sympy_: Call sympy_init

[sagetrac-git]

Changed commit from eef604e to 2baae58

[tscrim]

comment:8

I know it is not so useful, but please add a doctest for __new__. Perhaps running a TestSuite? Other than that, LGTM assume a green bot.

[sagetrac-git]

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

c06c965

sage.interfaces.sympy_wrapper.SageSet: Add another doctest

[sagetrac-git]

Changed commit from 2baae58 to c06c965

[tscrim]

comment:10

Thank you. Now once the bot comes back green, positive review.

[tscrim]

Reviewer: Travis Scrimshaw

[mkoeppe]

comment:13

Thanks!

[vbraun]

Changed branch from u/mkoeppe/wrapper_class_for_sage_sets_as_sympy_sets to c06c965