Skip to content
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

HPC-GAP: sync coll.gd and type.g #2501

Merged
merged 3 commits into from
Jun 1, 2018

Conversation

fingolfin
Copy link
Member

This subsumes PR #2500.

It is necessary for PR #2387, which otherwise would introduce diffs between GAP and HPC-GAP.

@fingolfin fingolfin added topic: HPC-GAP Issues and PRs related to HPC-GAP topic: HPC-GAP unification release notes: added PRs introducing changes that have since been mentioned in the release notes labels May 30, 2018
@codecov
Copy link

codecov bot commented May 30, 2018

Codecov Report

Merging #2501 into master will decrease coverage by <.01%.
The diff coverage is 80.76%.

@@            Coverage Diff             @@
##           master    #2501      +/-   ##
==========================================
- Coverage   74.26%   74.25%   -0.01%     
==========================================
  Files         484      482       -2     
  Lines      245343   245204     -139     
==========================================
- Hits       182208   182081     -127     
+ Misses      63135    63123      -12
Impacted Files Coverage Δ
lib/object.gd 50% <ø> (ø) ⬆️
lib/type.g 65.87% <ø> (ø) ⬆️
lib/coll.gd 91.41% <80.76%> (-2.12%) ⬇️
src/objset.c 84.59% <0%> (-0.23%) ⬇️
hpcgap/lib/hpc/stdtasks.g 64.7% <0%> (+0.76%) ⬆️

... they are used by HPC-GAP, and in GAP do nothing different than their
counterparts, namely:
  IsNonAtomicComponentObjectRep <-> IsComponentObjectRep
  IsReadOnlyPositionalObjectRep <-> IsPositionalObjectRep
  IsAtomicPositionalObjectRep   <-> IsPositionalObjectRep

We mainly provide them in plain GAP to make it easier to write code
that works well in both GAP and HPC-GAP.
Copy link
Contributor

@ssiccha ssiccha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ChrisJefferson ChrisJefferson merged commit 428c96f into gap-system:master Jun 1, 2018
@fingolfin fingolfin deleted the mh/hpcgap-sync2 branch June 1, 2018 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release notes: added PRs introducing changes that have since been mentioned in the release notes topic: HPC-GAP unification topic: HPC-GAP Issues and PRs related to HPC-GAP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants