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

WIP: Add IsDirectProductDomain objects and related basic methods #3477

Closed

Conversation

ssiccha
Copy link
Contributor

@ssiccha ssiccha commented May 27, 2019

lib: add IsDirectProductDomain objects and ...

... some related functions, operations, attributes, and methods.

TODO

  • doc
  • examples

Creates new files lib/productdomain.g{d,i}, tst/testinstall/productdomain.tst.
Moves the declaration, definition, and tests of DirectProductFamily from
lib/tuples.g{d,i} and hpcgap/lib/tuples.gi into the new files.

Adds the new files to doc/ref/makedocreldata.g.

Adds:

  • a filter IsDirectProductDomain
  • an operation DirectProductDomain to create IsDirectProductDomain
    domains and methods for DirectProductDomain
  • attributes and methods for IsDirectProductDomain objects:
    • ComponentsOfDirectProductDomain, this defines the domain
    • DimensionOfDirectProductDomain, the number of components
    • PrintObj
    • Size
    • \in

@ssiccha ssiccha added kind: new feature release notes: to be added PRs introducing changes that should be (but have not yet been) mentioned in the release notes topic: library labels May 27, 2019
@ssiccha ssiccha changed the title WIP: Add IsDirectProductDomain objects related basic functions WIP: Add IsDirectProductDomain objects and related basic methods May 27, 2019
@ssiccha
Copy link
Contributor Author

ssiccha commented May 27, 2019

This PR includes the three commits from PRs #3473, #3468, #3455 in order for the tests to work.

I still need to write documentation for IsDirectProductDomain and DirectProductDomain but everything else should be finished.

I didn't add implementations for Enumerator and AsList to this commit because they became a bit unwieldy. I'll open a seperate PR for those.

@ssiccha
Copy link
Contributor Author

ssiccha commented May 27, 2019

My goal is to be able to define direct products of mappings. For this I need to be able to construct direct products of domains first.

@ssiccha ssiccha added the do not merge PRs which are not yet ready to be merged (e.g. submitted for discussion, or test results) label May 27, 2019
@ssiccha ssiccha force-pushed the ss/add-DirectProductDomain branch from a899104 to ea4aa40 Compare May 27, 2019 17:18
@codecov
Copy link

codecov bot commented May 27, 2019

Codecov Report

Merging #3477 into master will increase coverage by 0.88%.
The diff coverage is 98.85%.

@@            Coverage Diff             @@
##           master    #3477      +/-   ##
==========================================
+ Coverage   84.54%   85.43%   +0.88%     
==========================================
  Files         698      701       +3     
  Lines      345286   346850    +1564     
==========================================
+ Hits       291937   296344    +4407     
+ Misses      53349    50506    -2843
Impacted Files Coverage Δ
lib/productdomain.gd 100% <100%> (ø)
lib/mapping.gi 89.78% <100%> (+1.17%) ⬆️
lib/read5.g 100% <100%> (ø) ⬆️
lib/read3.g 100% <100%> (ø) ⬆️
lib/productdomain.gi 98.64% <98.64%> (ø)
lib/csetpc.gi 42.69% <0%> (-45.51%) ⬇️
lib/autsr.gi 36.47% <0%> (-18.89%) ⬇️
lib/gpfpiso.gi 66.71% <0%> (-11.51%) ⬇️
lib/fieldfin.gi 78.3% <0%> (-10.2%) ⬇️
src/permutat.h 83.87% <0%> (-5.61%) ⬇️
... and 285 more

@ssiccha
Copy link
Contributor Author

ssiccha commented May 27, 2019

Something looks weird for the HPC-GAP tests. Fixed.

@ssiccha ssiccha force-pushed the ss/add-DirectProductDomain branch from ea4aa40 to 63fe2c9 Compare May 28, 2019 07:54
@coveralls
Copy link

coveralls commented May 28, 2019

Coverage Status

Coverage increased (+0.003%) to 85.262% when pulling f78ad27 on ssiccha:ss/add-DirectProductDomain into 597c793 on gap-system:master.

@ssiccha ssiccha force-pushed the ss/add-DirectProductDomain branch from 63fe2c9 to 13d0b46 Compare May 29, 2019 08:00
@ssiccha ssiccha force-pushed the ss/add-DirectProductDomain branch from 13d0b46 to 8be2984 Compare June 3, 2019 07:59
@ssiccha ssiccha force-pushed the ss/add-DirectProductDomain branch 2 times, most recently from 97c6d0c to f78ad27 Compare June 24, 2019 12:41
... some related functions, operations, attributes, and methods.

TODO
- doc
- examples
- maybe put the "move DirectProductFamily around" part into its own
  commit?

Creates new files `lib/productdomain.g{d,i}`, `tst/testinstall/productdomain.tst`.
Moves the declaration, definition, and tests of `DirectProductFamily` from
`lib/tuples.g{d,i}` and `hpcgap/lib/tuples.gi` into the new files.

Adds the new files to `doc/ref/makedocreldata.g` and to `read3.g` and
`read5.g`.

Adds:
- a filter IsDirectProductDomain
- an operation DirectProductDomain to create IsDirectProductDomain
  domains and methods for DirectProductDomain
- attributes and methods for IsDirectProductDomain objects:
  - ComponentsOfDirectProductDomain, this defines the domain
  - DimensionOfDirectProductDomain, the number of components
  - PrintObj
  - Size
  - \in
@ssiccha ssiccha force-pushed the ss/add-DirectProductDomain branch from f78ad27 to 0f86f3c Compare August 7, 2019 08:58
@ssiccha ssiccha closed this Feb 29, 2020
@ssiccha ssiccha deleted the ss/add-DirectProductDomain branch February 29, 2020 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge PRs which are not yet ready to be merged (e.g. submitted for discussion, or test results) kind: new feature release notes: to be added PRs introducing changes that should be (but have not yet been) mentioned in the release notes topic: library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants