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

pari call on x+O(x**2) fails #29293

Closed
fchapoton opened this issue Mar 7, 2020 · 9 comments
Closed

pari call on x+O(x**2) fails #29293

fchapoton opened this issue Mar 7, 2020 · 9 comments

Comments

@fchapoton
Copy link
Contributor

as follows:

sage: pari('x+O(x^2)')
---------------------------------------------------------------------------
PariError                                 Traceback (most recent call last)
<ipython-input-29-3844ab2b33fc> in <module>()
----> 1 pari('x+O(x^2)')

cypari2/pari_instance.pyx in cypari2.pari_instance.Pari.__call__()

cypari2/gen.pyx in cypari2.gen.objtogen()

cypari2/convert.pyx in cypari2.convert.PyObject_AsGEN()

cypari2/handle_error.pyx in cypari2.handle_error._pari_err_handle()

PariError: O(_^_): incorrect type in O (t_SER)

whereas it works in gp:

sage: %gp

  --> Switching to PARI/GP interpreter <--

pari: x+O(x^2)
x + O(x^2)

This prevents automatic testing of PARI code in oeis.

EDIT:

But this works:

sage: pari('x+O(x)^2')
x + O(x^2)

CC: @tscrim

Component: interfaces

Keywords: pari

Reviewer: Travis Scrimshaw

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

@fchapoton fchapoton added this to the sage-9.1 milestone Mar 7, 2020
@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

@mkoeppe
Copy link
Member

mkoeppe commented Mar 12, 2020

comment:3

I also see a failure in the cypari testsuite that looks similar

@fchapoton
Copy link
Contributor Author

comment:4

Hmm, it seem to works for me now. Invalid ticket ?

@fchapoton
Copy link
Contributor Author

comment:5

Travis, could you please double check that this works and the ticket is invalid ?

By the way, how do you do ? Are you confined as I am ?

@tscrim
Copy link
Collaborator

tscrim commented Mar 24, 2020

comment:6

Replying to @fchapoton:

Travis, could you please double check that this works and the ticket is invalid ?

Yes, it now works. Since it is upstream in cypari2, I don't think we need to (or should) add a doctest.

By the way, how do you do ? Are you confined as I am ?

I am doing okay. I am confined to Australia since they closed their boarders and there are essentially no international flights from Australia. However I am still able to go outside and to my office at UQ. However, the situation seems to change daily without much warning, so who knows how long that will last.

@tscrim
Copy link
Collaborator

tscrim commented Mar 24, 2020

Reviewer: Travis Scrimshaw

@tscrim tscrim removed this from the sage-9.1 milestone Mar 24, 2020
@tscrim
Copy link
Collaborator

tscrim commented Mar 24, 2020

comment:7

How are you doing?

@fchapoton
Copy link
Contributor Author

comment:8

I going well, as far as possible.

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