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

triangle inequality for naturals and integers #177

Merged

Conversation

lane-core
Copy link
Contributor

On Dr. Escardo's invitation from mathstodon, I am contributing my proof of the triangle inequality in the Naturals and the Integers. In the Naturals, I define a new Absolute Difference operation and prove some elementary lemmas. I then add associated lemmas for Absolute Difference to the Natural Order lemmas, proving the triangle inequality on the way for the naturals under this operation. I then prove some useful lemmas for Absolute Value in the Integers related to this new operation, and conclude by proving the triangle inequality in the Integer Order file (for both abs and adding a translated lemma to absZ for convenience).

@martinescardo
Copy link
Owner

Thanks for your contribution. Will have a look now.

Copy link
Owner

@martinescardo martinescardo left a comment

Choose a reason for hiding this comment

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

Nice! Thanks for your contribution. I will accept your pull request as it is, but please address the formatting problems in a second pull request after this.

source/Integers/Abs.lagda Show resolved Hide resolved
source/Integers/Abs.lagda Show resolved Hide resolved
source/Integers/Order.lagda Show resolved Hide resolved
source/Integers/Order.lagda Show resolved Hide resolved
@martinescardo
Copy link
Owner

Waiting for the checks to complete to merge.

@martinescardo
Copy link
Owner

You should also add yourself to the list of contributors in the top-level README file.

@martinescardo
Copy link
Owner

You have added the year of your contribution. Please also add the day and month.

@martinescardo martinescardo merged commit 72e37ac into martinescardo:master Sep 7, 2023
1 check passed
@martinescardo
Copy link
Owner

When you have a chance, please to a pull request to address the above issues. Thanks.

@lane-core
Copy link
Contributor Author

@martinescardo I will work on this clean-up over the weekend, thank you for accepting my work!

@lane-core
Copy link
Contributor Author

Done with all these, I submitted a new PR to amend all these at the end of last week. Thank you!

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.

2 participants