Opened 13 years ago

Closed 12 years ago

#5467 closed defect (fixed)

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

Reported by: was Owned by: mhansen
Priority: major Milestone: sage-4.5.2
Component: interfaces Keywords:
Cc: Merged in: sage-4.5.2.alpha0
Authors: Andrey Novoseltsev Reviewers: Mike Hansen
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by novoselt)

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 (2)

5467_long_input_to_Macaulay2_second_part.patch (750 bytes) - added by novoselt 12 years ago.
Must be applied after the first patch.
5467_long_input_to_Macaulay2.patch (1.7 KB) - added by novoselt 12 years ago.
Only this patch should be applied.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 13 years ago by mabshoff

  • Milestone changed from sage-3.4 to sage-3.4.1

comment:2 Changed 12 years ago by novoselt

  • Authors set to Andrey Novoseltsev
  • Report Upstream set to N/A
  • Status changed from new to needs_review

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 12 years ago by novoselt

Must be applied after the first patch.

comment:3 Changed 12 years 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.

comment:4 Changed 12 years ago by novoselt

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

Changed 12 years ago by novoselt

Only this patch should be applied.

comment:5 Changed 12 years ago by novoselt

  • Description modified (diff)

comment:6 Changed 12 years ago by mhansen

  • Reviewers set to Mike Hansen
  • Status changed from needs_review to positive_review

Looks good to me.

comment:7 Changed 12 years ago by mpatel

  • Merged in set to sage-4.5.2.alpha0
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.