gmpy2.mpq can not be initialized from a pair of Sage integers

[videlec]

works fine from integers

sage: gmpy2.mpq(12r)
mpq(12,1)
sage: gmpy2.mpq(12)
mpq(12,1)

but not for pairs

sage: gmpy2.mpq(2r,3r)
mpq(2,3)
sage: gmpy2.mpq(2,3)
Traceback (most recent call last):
...
TypeError: cannot convert object to mpq

upstream issue : aleaxit/gmpy#237

Depends on #27858

Upstream: Reported upstream. Developers acknowledge bug.

CC: @vinklein

Component: packages: standard

Keywords: thursdaysbdx

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

[vinklein]

comment:1

Indeed.

And it's clearly a gmpy2 issue :

>>> from gmpy2 import mpz
>>> import gmpy2
>>> class One:
...     def __mpz__(self): return mpz(1)
... 
>>> class Two:
...     def __mpz__(self): return mpz(2)
... 
>>> gmpy2.mpq(Two())
mpq(2,1)
>>> gmpy2.mpq(One(), Two())
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: cannot convert object to mpq

I will look at this.

[vinklein]

Upstream: Reported upstream. No feedback yet.

[vinklein]

Changed upstream from Reported upstream. No feedback yet. to Reported upstream. Developers acknowledge bug.

[vinklein]

comment:3

As the next beta release is coming soon i don't think a patch is necessary

[vinklein]

Changed keywords from none to thursdaysbdx

[videlec]

comment:4

Good. Thank you of taking care of it!

[vinklein]

Dependencies: #27858

[vinklein]

comment:6

No point in keeping this ticket.
Unless one want to add some doctests to remember that the problem is fixed.