Opened 2 years ago

Closed 2 years ago

#28571 closed enhancement (fixed)

pass 'macaulay2' as argument to '_macaulay2_init_()'

Reported by: gh-mwageringel Owned by:
Priority: major Milestone: sage-9.0
Component: interfaces: optional Keywords: macaulay2, IMA Coding Sprint
Cc: saliola, dimpase, gh-antonleykin Merged in:
Authors: Markus Wageringel Reviewers: Franco Saliola
Report Upstream: N/A Work issues:
Branch: 2823dc0 (Commits, GitHub, GitLab) Commit: 2823dc0abc2ed82b731922ef0492dc37406aba9a
Dependencies: #28074 Stopgaps:

Status badges

Description

This gives access to the Macaulay2 interpreter in the _macaulay2_init_() function, so it can also return Macaulay2Elements rather than just strings. The _macaulay2_init_() function is used for initializing Macaulay2 elements and is usually called by _macaulay2_() which then takes care of caching.

The magma interface also does this, but it had not been correctly set up until now.

Change History (5)

comment:1 Changed 2 years ago by gh-mwageringel

  • Authors set to Markus Wageringel
  • Branch set to public/interfaces/m2/macaulay2_init
  • Commit set to 2823dc0abc2ed82b731922ef0492dc37406aba9a
  • Dependencies set to #28074

I have not tested the change for Magma as I do not have access to it.

---

New commits:

5709e6128074: refactor caching of Macaulay2 polynomial rings
c2d89a0Merge tag '9.0.beta0'
2823dc0pass macaulay2 as argument to _macaulay2_init_()
Last edited 2 years ago by gh-mwageringel (previous) (diff)

comment:2 Changed 2 years ago by saliola

  • Status changed from new to needs_review

comment:3 Changed 2 years ago by saliola

  • Status changed from needs_review to positive_review

comment:4 Changed 2 years ago by gh-mwageringel

  • Reviewers set to Franco Saliola

comment:5 Changed 2 years ago by vbraun

  • Branch changed from public/interfaces/m2/macaulay2_init to 2823dc0abc2ed82b731922ef0492dc37406aba9a
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.