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

Non-distributive lattice structures #453

Merged
merged 2 commits into from Jan 30, 2020

Conversation

pi8027
Copy link
Member

@pi8027 pi8027 commented Jan 9, 2020

Motivation for this change

This PR adds non-distributive lattice structures latticeType, bLatticeType, tbLatticeType, and finLatticeType, and big pack notations suggested by @CohenCyril. This PR was originally opened as PR #388, needed to be recreated to change the target branch from experiment/order to master.

Things done/to do
  • added corresponding entries in CHANGELOG_UNRELEASED.md
  • added corresponding documentation in the headers
Automatic note to reviewers

Read this Checklist and make sure there is a milestone.

@pi8027
Copy link
Member Author

pi8027 commented Jan 9, 2020

@strub Can you be the assignee?

@strub
Copy link
Member

strub commented Jan 9, 2020

Sure

@strub
Copy link
Member

strub commented Jan 13, 2020

Can somebody assigned me?

@pi8027
Copy link
Member Author

pi8027 commented Jan 14, 2020

@strub @CohenCyril Thank you.

@pi8027 pi8027 force-pushed the experiment/order-nondistr-lattice branch from 547eb6c to 00f593f Compare January 15, 2020 11:30
@strub
Copy link
Member

strub commented Jan 27, 2020

I don't think I have that much to say about this PR. I am now using it for 1 month without any problem and the diff seems fine by me.

IMO, it is ready for merge.

Copy link
Member

@strub strub left a comment

Choose a reason for hiding this comment

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

@pi8027 Could you add an entry in CHANGELOG_UNRELEASED.md?

@pi8027
Copy link
Member Author

pi8027 commented Jan 29, 2020

@strub Yes, I will do that. Since #270 has not been released yet, it can be a small change. But I also should make the header consistent with implementation.

@pi8027
Copy link
Member Author

pi8027 commented Jan 29, 2020

Done. I think this should be OK, but I can wait for more feedback.

@strub strub merged commit 7d04173 into math-comp:master Jan 30, 2020
@pi8027 pi8027 deleted the experiment/order-nondistr-lattice branch January 30, 2020 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants