Opened 3 years ago

Last modified 3 years ago

#25273 closed enhancement

Unprefixed libGAP interface — at Version 9

Reported by: jdemeyer Owned by:
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: interfaces Keywords:
Cc: slelievre Merged in:
Authors: Jeroen Demeyer Reviewers:
Report Upstream: N/A Work issues:
Branch: u/jdemeyer/unprefixed_libgap_interface (Commits, GitHub, GitLab) Commit: ea2e39c7374b03b97366f264c89a0f4c75a7c172
Dependencies: #25274 Stopgaps:

Status badges

Description (last modified by dimpase)

Use Cython cname declarations to use unprefixed libGAP names in Sage. The libGAP package is not changed at all, only the Sage interface.

Obvious exceptions:

  • libGAP_True
  • libGAP_False

Change History (9)

comment:1 Changed 3 years ago by jdemeyer

  • Description modified (diff)

comment:2 Changed 3 years ago by jdemeyer

  • Description modified (diff)

comment:3 Changed 3 years ago by jdemeyer

  • Dependencies set to #25274

comment:4 Changed 3 years ago by dimpase

  • Dependencies #25274 deleted

Does this mean that libgap.Blah syntax won't be needed, too?

comment:5 Changed 3 years ago by dimpase

  • Dependencies set to #25274

comment:6 Changed 3 years ago by jdemeyer

  • Branch set to u/jdemeyer/unprefixed_libgap_interface

comment:7 Changed 3 years ago by jdemeyer

  • Commit set to ea2e39c7374b03b97366f264c89a0f4c75a7c172
  • Status changed from new to needs_review

New commits:

b8c3c12Clean up cimports in libgap
ea2e39cUnprefixed libGAP interface

comment:8 Changed 3 years ago by jdemeyer

  • Description modified (diff)

comment:9 Changed 3 years ago by dimpase

  • Description modified (diff)

In libgap.pyx you have

The ``foobar`` methods are
the original GAP methods simply prefixed with the string
````.

which looks like an artefact of non-interactive editing...

I presume there is no prefix any more.

Note: See TracTickets for help on using tickets.