Skip to content

Conversation

@BookWood7th
Copy link
Contributor

This PR adds documentation for the Isabelle Translation extension proposed in (KeYProject/key#3514).

Copy link
Member

@WolframPfeifer WolframPfeifer left a comment

Choose a reason for hiding this comment

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

Thanks for writing a nice user documentation of your feature!

I think it would make sense to add the following points:

  • recommendation: In contrast to SMT solvers, it will not be a good idea to have more than 1 or maybe 2 instances at the same time.
  • possible plans for the future (but not possible at the moment): interactive proving

I was also wondering whether it would be useful to have a description (one subsection, very high-level) of how the translation works (with the locale, type encoding, and all that). Just to provide a rough idea of what is actually done, without having to read through the thesis.

I would wait with merge until the feature is merged into KeY ...

@WolframPfeifer
Copy link
Member

Can be merged as soon as KeYProject/key#3514 is merged into KeY. Thanks!

@WolframPfeifer WolframPfeifer merged commit 3f58681 into KeYProject:master May 19, 2025
2 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