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

ENHANCE: Option NoPrecomputedData to avoid use of data libraries #986

Merged
merged 7 commits into from
Dec 15, 2016

Conversation

hulpke
Copy link
Contributor

@hulpke hulpke commented Dec 6, 2016

Tick all what applies to this pull request

  • Adds new features
  • Improves and extends functionality

Write below the description of changes (for the release notes)

Add the function option NoPrecomputedData to turn off reliance on data libraries. This applies for all uses of Small groups/Tom/Prim/Trans data libraries in the main GAP library.

Added deocumentation in section on data libraries.

@hulpke hulpke added the do not merge PRs which are not yet ready to be merged (e.g. submitted for discussion, or test results) label Dec 6, 2016
@codecov-io
Copy link

codecov-io commented Dec 6, 2016

Current coverage is 49.45% (diff: 73.21%)

Merging #986 into master will decrease coverage by 0.01%

@@             master       #986   diff @@
==========================================
  Files           424        424          
  Lines        223224     223260    +36   
  Methods        3430       3430          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits         110418     110409     -9   
- Misses       112806     112851    +45   
  Partials          0          0          

Powered by Codecov. Last update 70ed8c5...809b20c

@fingolfin
Copy link
Member

Seems plausible to me.

@hungaborhorvath
Copy link
Contributor

Some of these commits are already in master. Maybe an accidental merge happened instead of a rebase?

@hulpke
Copy link
Contributor Author

hulpke commented Dec 9, 2016

@hungaborhorvath
I don't think so -- what seems to be rather happening is that a rebase included several commits in master again as part of this branch. I'll see to fix this.

This is only supposed to be merged after recent changes in Additions branch

@hulpke hulpke removed the do not merge PRs which are not yet ready to be merged (e.g. submitted for discussion, or test results) label Dec 14, 2016
@hulpke hulpke merged commit 6e5d6ea into gap-system:master Dec 15, 2016
@olexandr-konovalov olexandr-konovalov added the release notes: added PRs introducing changes that have since been mentioned in the release notes label Jan 20, 2018
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants