Opened 13 years ago

Last modified 6 weeks ago

#9503 new defect

FreeModule_submodule_with_basis_pid calls wrong constructor

Reported by: novoselt Owned by: AlexGhitza
Priority: major Milestone:
Component: algebra Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

GitHub link to the corresponding issue

Description

This is a piece of the current code in FreeModule_submodule_with_basis_pid after #9502 (before it was the same without explanations)

# The following is WRONG - we should call __init__ of
# FreeModule_generic_pid. However, it leads to a bunch of errors.
FreeModule_generic.__init__(self, R,
                            rank=len(basis), degree=ambient.degree(),
                            sparse=ambient.is_sparse())

The errors seem to be related to number fields and their rings of integers.

Change History (5)

comment:1 Changed 9 years ago by jdemeyer

Milestone: sage-5.11sage-5.12

comment:2 Changed 9 years ago by vbraun_spam

Milestone: sage-6.1sage-6.2

comment:3 Changed 9 years ago by vbraun_spam

Milestone: sage-6.2sage-6.3

comment:4 Changed 9 years ago by vbraun_spam

Milestone: sage-6.3sage-6.4

comment:5 Changed 6 weeks ago by mkoeppe

Milestone: sage-6.4
Note: See TracTickets for help on using tickets.