Ticket #11390 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

NTL error while creating finite field homomorphism

Reported by: jpflori Owned by: AlexGhitza
Priority: blocker Milestone: sage-4.7.1
Component: algebra Keywords:
Cc: SimonKing, jpflori, malb Work issues:
Report Upstream: N/A Reviewers: Simon King, Martin Albrecht
Authors: Jean-Pierre Flori Merged in: sage-4.7.1.alpha2
Dependencies: Stopgaps:

Description (last modified by jdemeyer) (diff)

On my computer I get an unhandled NTL error by just doing:

sage: K = GF(1<<16,'a'); L = GF(1<<32,'b')
sage: K.Hom(L)[0]

See  http://groups.google.com/group/sage-support/browse_thread/thread/b9714926d491e905 for further description and potential fix.

I'm setting it as blocker as Simon suggested, feel free to change.

There seem to be some initialization problem with NTL and GF2X.

Attachments

trac_11390-fix.patch Download (706 bytes) - added by jpflori 2 years ago.
Potential fix
trac_11390-doctest.patch Download (1.1 KB) - added by jpflori 2 years ago.
Doctest

Change History

Changed 2 years ago by jpflori

Potential fix

comment:1 Changed 2 years ago by jpflori

I'll provide another patch for doctesting the problem in a few moments.

Changed 2 years ago by jpflori

Doctest

comment:2 Changed 2 years ago by jpflori

  • Status changed from new to needs_review

comment:3 Changed 2 years ago by SimonKing

  • Cc malb added

It seems to me that the problem is solved. However, if the original author (Martin Albrecht, I just cc'ed him) has time to look at you patch, it might be better to know his opinion first.

comment:4 Changed 2 years ago by malb

  • Status changed from needs_review to positive_review
  • Reviewers set to Simon King, Martin Albrecht

Patch looks good.

comment:5 Changed 2 years ago by jdemeyer

  • Description modified (diff)
  • Summary changed from Segfault while creating finite field homomorphism to NTL error while creating finite field homomorphism

comment:6 Changed 2 years ago by jdemeyer

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