-
Notifications
You must be signed in to change notification settings - Fork 161
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use LoadKernelExtension
and IsKernelExtensionAvailable
in more packages
#5761
Use LoadKernelExtension
and IsKernelExtensionAvailable
in more packages
#5761
Comments
Some of us at GAPDays are going to work on this today. |
I'm going to make a start on |
I will take |
Working on |
Working on |
Working on |
Thanks for the efforts, folks. A general comment: I think it is fine to just change all these packages to require GAP >= 4.12. After all: not many people will update a single package; most just update all of GAP at once. (This might change in the future if PackageManager gets more widely used but I don't think we are there yet, so we don't need to worry about it. If you do it that way, then also update |
@fingolfin Lets take it as definition of done, if so. Pull-request: homalg-project/homalg_project#614. |
Doing float now |
Just to say: it might be a bit premature to mark a package as "fixed" in the list at the top of this issue right after the corresponding PR was merged: in my eyes it is only truly "fixed" if there also is a release of that package with the changes. |
These function have been available since GAP 4.12.0 and I think we should use them in more (ideally all) packages with kernel extension. So far only
edim
andbrowse
do so.Here is a list of affected distributed packages. The goal would be to either submit an issue or a PR to each of these over time.
cddinterface
: Resolve https://github.com/gap-system/gap/issues/5761 homalg-project/CddInterface#43, done in v2024.08.27crypting
: Resolve https://github.com/gap-system/gap/issues/5761 gap-packages/crypting#24, done in v0.10.5curlinterface
: Resolve https://github.com/gap-system/gap/issues/5761 gap-packages/curlInterface#52, done in v2.4.0cvec
: Update kernel module checking gap-packages/cvec#45datastructures
: Resolve https://github.com/gap-system/gap/issues/5761 gap-packages/datastructures#153deepthought
: Resolve https://github.com/gap-system/gap/issues/5761 gap-packages/DeepThought#21digraphs
: Load kernel ext digraphs/Digraphs#668, done in v1.9.0ferret
: Update kernel module checking gap-packages/ferret#61float
: Update kernel module checking gap-packages/float#94gauss
: Add LoadKernelExtension support homalg-project/homalg_project#614hap
: ImplementLoadKernelExtension
gap-packages/hap#123io
: Resolve https://github.com/gap-system/gap/issues/5761 gap-packages/io#125, done in v4.9.0json
: Add LoadKernelExtension gap-packages/json#33normalizinterface
: Require GAP >= 4.12.1; Use LoadKernelExtension/IsKernelExtensionAvailable gap-packages/NormalizInterface#117orb
: Add LoadKernelExtension support gap-packages/orb#69profiling
: Update kernel module checking gap-packages/profiling#118semigroups
: Better kernel module checking semigroups/Semigroups#1032zeromqinterface
: https://github.com/gap-packages/ZeroMQInterface/pullsThe text was updated successfully, but these errors were encountered: