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

Declare Integer.value as array #24259

Closed
jdemeyer opened this issue Nov 21, 2017 · 8 comments
Closed

Declare Integer.value as array #24259

jdemeyer opened this issue Nov 21, 2017 · 8 comments
Labels
c: cython p: major / 3 t: bug
Milestone
sage-8.1

Comments

@jdemeyer
Copy link

Work around cython/cython#1984

Upstream: Reported upstream. No feedback yet.

Component: cython

Author: Jeroen Demeyer

Branch/Commit: 88eaed5

Reviewer: Marc Mezzarobba

Issue created by migration from https://trac.sagemath.org/ticket/24259
@jdemeyer jdemeyer added this to the sage-8.1 milestone Nov 21, 2017
@jdemeyer
Copy link
Author

Branch: u/jdemeyer/declare_integer_value_as_array

@jdemeyer
Copy link
Author

New commits:

88eaed5Declare Integer.value as array

@jdemeyer
Copy link
Author

Commit: 88eaed5

@jdemeyer
Copy link
Author

Author: Jeroen Demeyer

@mezzarobba
Copy link
Member

comment:4

Why aren't other gmp/mpfr/flint... structures affected?

@jdemeyer
Copy link
Author

comment:5

Replying to @mezzarobba:

Why aren't other gmp/mpfr/flint... structures affected?

They are all affected equally. It's just that I'm only using Integer internals in such a way that I encounter the Cython bug.

@mezzarobba
Copy link
Member

Reviewer: Marc Mezzarobba

@vbraun
Copy link
Member

vbraun commented Dec 13, 2017

Changed branch from u/jdemeyer/declare_integer_value_as_array to 88eaed5

@vbraun vbraun closed this as completed in 5fb5ca3 Dec 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: cython p: major / 3 t: bug
Projects
None yet
Milestone
sage-8.1
Development

No branches or pull requests
3 participants
@vbraun @jdemeyer @mezzarobba