Opened 3 years ago

Last modified 4 months ago

#25558 new enhancement

Move multi-variate polynomials to new coercion model

Reported by: jdemeyer Owned by:
Priority: critical Milestone: sage-9.5
Component: coercion Keywords:
Cc: andrew.mathas, tscrim, gh-mwageringel Merged in:
Authors: Jeroen Demeyer Reviewers:
Report Upstream: N/A Work issues:
Branch: u/jdemeyer/ticket/25558 (Commits, GitHub, GitLab) Commit: 160624fc77a13d4a528a7e7eeaa73fc10b562d54
Dependencies: Stopgaps:

Status badges

Description


Change History (23)

comment:1 Changed 3 years ago by jdemeyer

  • Dependencies set to #25555

comment:2 Changed 3 years ago by jdemeyer

  • Branch set to u/jdemeyer/ticket/25558

comment:3 Changed 3 years ago by jdemeyer

  • Commit set to d6b2d6acb97133cabf6a5c4b1f06705ef65f0edf
  • Dependencies changed from #25555 to #25555, #25563

New commits:

e1bb9b6Replace _coerce_ checks by has_coerce_map_from
d6b2d6aMove multi-variate polynomials to new coercion model

comment:4 Changed 3 years ago by git

  • Commit changed from d6b2d6acb97133cabf6a5c4b1f06705ef65f0edf to 2717d6ef96411c9d9600725e781b40a969b9e848

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

a3559d8Rename MPolynomialRing_generic -> MPolynomialRing_base
2717d6eMove multi-variate polynomials to new coercion model

comment:5 Changed 3 years ago by andrew.mathas

  • Cc Andrew Mathas added

comment:6 Changed 3 years ago by andrew.mathas

  • Cc andrew.mathas added; Andrew Mathas removed

comment:7 Changed 3 years ago by git

  • Commit changed from 2717d6ef96411c9d9600725e781b40a969b9e848 to 160624fc77a13d4a528a7e7eeaa73fc10b562d54

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

160624fMove multi-variate polynomials to new coercion model

comment:8 Changed 3 years ago by jdemeyer

  • Dependencies #25555, #25563 deleted

comment:9 Changed 3 years ago by tscrim

  • Cc tscrim added

comment:10 Changed 3 years ago by etn40ff

In #26741 Jeroen mentioned that he might be too busy to work on this full time.

If the stars align properly I will try to be at "Calcul Mathématique Libre" and I would be happy to work on this there but, as I said, I might need some guidance to do so. Would any of the attendees be willing/able to provide it?

comment:11 Changed 3 years ago by tscrim

I will not be there, but I can try to be available remotely to work on it.

comment:12 Changed 3 years ago by etn40ff

Thanks Travis, I hope I'll be able to be there.

comment:13 Changed 3 years ago by jdemeyer

  • Dependencies set to #27034
  • Milestone changed from sage-8.3 to sage-8.7

Said conference is this week but given that neither of you are here there is no real point in doing it right now.

comment:14 Changed 3 years ago by dkrenn

I am not sure what exactly will be done on this ticket, but FYI there was some low level stuff regarding *conversion* done at #21855 in src/sage/rings/polynomial/laurent_polynomial_ring.py (_split_dict_ functions).

(This comment is a result of mentioning this ticket here at #27255.)

comment:15 Changed 3 years ago by etn40ff

Sorry for flying off the radars for a while but I had an unexpected change of job. The dust is still settling but I should be able to resume working on this whenever you want to.

comment:16 Changed 3 years ago by embray

  • Milestone changed from sage-8.7 to sage-8.8

Ticket retargeted after milestone closed (if you don't believe this ticket is appropriate for the Sage 8.8 release please retarget manually)

comment:17 Changed 3 years ago by dkrenn

Are there any more concrete plans working on this ticket?

comment:18 Changed 2 years ago by embray

  • Milestone sage-8.8 deleted

As the Sage-8.8 release milestone is pending, we should delete the sage-8.8 milestone for tickets that are not actively being worked on or that still require significant work to move forward. If you feel that this ticket should be included in the next Sage release at the soonest please set its milestone to the next release milestone (sage-8.9).

comment:19 Changed 15 months ago by mkoeppe

  • Milestone set to sage-9.3

comment:20 Changed 10 months ago by mkoeppe

  • Milestone changed from sage-9.3 to sage-9.4

Setting new milestone based on a cursory review of ticket status, priority, and last modification date.

comment:21 Changed 8 months ago by mkoeppe

  • Cc gh-mwageringel added
  • Priority changed from major to critical

comment:22 Changed 6 months ago by vdelecroix

  • Dependencies #27034 deleted

comment:23 Changed 4 months ago by mkoeppe

  • Milestone changed from sage-9.4 to sage-9.5
Note: See TracTickets for help on using tickets.