Ticket #5467 (needs_review defect)

Opened 12 months ago

Last modified 5 weeks ago

The sage/macaulay2 interface doesn't work at all on large input

Reported by: was Owned by: mhansen
Priority: major Milestone: sage-4.3.4
Component: interfaces Keywords:
Cc: Author(s): Andrey Novoseltsev
Report Upstream: N/A Reviewer(s):
Merged in: Work issues:

Description (last modified by novoselt) (diff)

ON OS X and Linux with Macaulay 1.2.

teragon:~ wstein$ sage
----------------------------------------------------------------------
| Sage Version 3.4.alpha0, Release Date: 2009-02-24                  |
| Type notebook() for the GUI, and license() for information.        |
----------------------------------------------------------------------
sage: macaulay2(10^10000)
sage0
sage: macaulay2(10^10000)
sage1
sage: macaulay2(10^10000)
sage2

See also #7897 and #7915.

Attachments

5467_long_input_to_Macaulay2_second_part.patch Download (0.7 KB) - added by novoselt 2 months ago.
Must be applied after the first patch.
5467_long_input_to_Macaulay2.patch Download (1.7 KB) - added by novoselt 5 weeks ago.
Only this patch should be applied.

Change History

Changed 12 months ago by mabshoff

  • milestone changed from sage-3.4 to sage-3.4.1

Changed 2 months ago by novoselt

  • status changed from new to needs_review
  • upstream set to N/A
  • author set to Andrey Novoseltsev

This is a trivial fix - the problem was in a wrong file loading command passed to Macaulay2.

The tests in the modified function pass (although optional tests for the whole file give a bunch of unrelated mistakes).

Changed 2 months ago by novoselt

Must be applied after the first patch.

Changed 2 months ago by novoselt

I managed to make a trivial mistake in this trivial fix - one of the test lines was not marked with #optional. Now it is.

Changed 5 weeks ago by novoselt

Converged patches into a single one and made the commit message of standard form.

Changed 5 weeks ago by novoselt

Only this patch should be applied.

Changed 5 weeks ago by novoselt

  • description modified (diff)
Note: See TracTickets for help on using tickets.