Update semidp.tst for compatibility with future GAP version #566
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I have created PR gap-system/gap#3053 for GAP, which adds additional properties that are transferred from the factors of a direct product to the direct product, at creation.
This means that, once this is merged, many direct products will know more about themselves at creation than they currently do. This means the test output in
semidp.tst
andsemidp.xml
needs to be updated. However, since we still need to support previous versions of GAP, we can't just update the files with the new output. We have to make sure that the properties are definitely computed before viewing the direct product, and showing the new output. That's why there are lots of new calls toIsRegularSemigroup
, etc.The tests on the
GAP stable-4.10, 64-bit
Travis job will fail until #565 is fixed and merged.