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

Problems with Sequent Display #35

Closed
davidfaraci opened this issue Aug 26, 2015 · 3 comments
Closed

Problems with Sequent Display #35

davidfaraci opened this issue Aug 26, 2015 · 3 comments
Labels

Comments

@davidfaraci
Copy link
Collaborator

Sequents do not display premises in stated order. Example:

image
Sequents sometimes display premises multiple times. Example:

image

gleachkr added a commit that referenced this issue Aug 26, 2015
@gleachkr
Copy link
Owner

Ok, I think I've fixed the repeated premise issue. The ordering of premises is tricky---it's determined basically by the form of the inference rules you use, I think, which can interact in complicated ways pretty quickly.

I think the easiest way to get the order of premises in the sequent to match the order of their occurrence in the proof would be to look the proof after figuring out what it established, and then to re-order the premises. This would involve a lot of code reorganization. I think it might be better to put this on hold until writing a new natural deduction module like the one projected in #23.

@davidfaraci
Copy link
Collaborator Author

Repeated premise issues seems to be fixed. Sounds like a good idea to wait on the other issue.

@gleachkr
Copy link
Owner

Ok. I'll add a note to #23

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants