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

Add preliminary support for First-Order Logic #140

Merged
merged 1 commit into from
Jan 21, 2024
Merged

Add preliminary support for First-Order Logic #140

merged 1 commit into from
Jan 21, 2024

Conversation

marcofavorito
Copy link
Member

@marcofavorito marcofavorito commented Jan 7, 2024

Proposed changes

This PR adds preliminary support for First-Order Logic and an implementation of natural deduction.

In particular, the added files in commit 3c49b23 are taken from the external repository paulotguerra/pylogics-extras@6848d5f, and the copyright holder is the author of that commit (Paulo T. Guerra paulodetarso@ufc.br (@paulotguerra)). The commit 3c49b23 is authored by @paulotguerra to recognize the authorship of the code (but it has been committed by @marcofavorito for practical purposes).

Paulo will be added as a contributor to the project and, therefore, will hold the copyright. The license will implicitly reflect this using the notice "Copyright (c) 2021-2024 The Pylogics contributors" (see #139). The fine-grained information on the contributions is stored in the Git history of the project.

The code needs relicensing of his code with the MIT license rather than the GPL (see pylogics-extras LICENSE). This is needed since after #139 the Pylogics' LICENSE is the MIT License, which is incompatible with GPL. This has been agreed upon in private communication with @paulotguerra, and he can confirm this licensing change on his code by commenting on this PR.

This commit is taken from the external repository paulotguerra/pylogics-extras@6848d5f,
and the copyright holder is the author of this commit: Paulo T. Guerra <paulodetarso@ufc.br> (@paulotguerra)
@paulotguerra
Copy link
Contributor

I do confirm the repective licensing change.

@marcofavorito marcofavorito self-assigned this Jan 8, 2024
@marcofavorito
Copy link
Member Author

marcofavorito commented Jan 21, 2024

Merging this now, we can continue working on this feature on separate PRs.

@marcofavorito marcofavorito merged commit 3f5f317 into main Jan 21, 2024
5 of 6 checks passed
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