Fix gap cyclotomics to sage
|Reported by:||tscrim||Owned by:||tscrim|
|Authors:||Travis Scrimshaw||Reviewers:||Frédéric Chapoton|
|Report Upstream:||N/A||Work issues:|
Description (last modified by tscrim)
Current gap expects a particular method of _n() which is provided by CyclotomicField but not by others, in particular UniversalCyclotomicField.
sage: libgap.E(3).sage(ring=UniversalCyclotomicField()) AttributeError: 'UniversalCyclotomicField_with_category' object has no attribute '_n'
This ticket fixes this by catching this exception and converting through the CyclotomicField.
Change History (8)
comment:1 Changed 7 months ago by tscrim
- Description modified (diff)
- Status changed from new to needs_review
Changed 7 months ago by tscrim
comment:5 Changed 7 months ago by chapoton
- Reviewers set to Frédéric Chapoton
- Status changed from needs_review to positive_review