Restyle Exploration of the Lower Your Guards paper #402
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![restyled-io[bot]](https://avatars.githubusercontent.com/in/5851?s=60&v=4){kind=small}
LeitMoth
added a commit
that referenced
this pull request
Sep 21, 2024
* Init stack project explore/lower-your-guards * lyg test language parser works * basic gaurd tree desugaring * add basic uncovered set function * add pair datacons and pretty printer * inhabitants N function kind of works * Add types * inhabitants mostly working * pretty print patterns * start work on fresh * Fresh works! * Fixed arg id bug, fresh actually works now! * implement Show to make some printing better * done for day. Strange behavoir, check complex2/3 * switch to MaybeT / cleanup * nested patterns work! Still must tune up Int literal stuff, also see type equal TODO * positive info works * positive info and integer negative info work! * no longer list all dataconstructors anyway when covered * remove old negative only inhab code * cleanup comments in inhab hs * remove old TODO message * change refinement type formulas datatype to match paper Originally, I thought I had outsmarted the paper Not so, they broke things up into `Formula`s and `Literal`s for a good reason, which I rediscovered * auto run on files in test folder * huge overhaul to Inhabitants.hs. more to go, but significant cleanup done * refactor / cleanup adding constraints * refactor addLiteral previously known as (<+>) for readability * rename matching to onVar to avoid confusion * major cleanup to inhabitants + lots of documentation * cleanup Pos suffix, all functions use positive info now * small inhabitant cleanup + ready for big changes * play with map merge instead of logic solver idea in inhab2 * add Annotated tree and redundancy checking * attempt my own version of exhaustiveness checking * BROKEN, about to try list monad * match tree broken, might be a lost cause * combined UA function done! * Properly use Fresh moand in Inhabitation checks Before I was manually threading state By rewriting a few things, Fresh and Monad [] can coexist Really hope there are no regressions, I didn't spot any * add comments, prepare for simplification * add throol testcase, proves necessity of cart prod * create Possibilities newtype for lists, to make things more clear * cleanup inhabitants * document + move Possibilities to a new file * attempting to remove MatchInt (CURRENTLY BROKEN) * add int as opaque type, not weird exception (BROKEN) * inhab works again! needs cleanup * remove traces of IntLit, ready for MatchInfo merge * introduce HerebyBe, reads better, hopefully no regressions * Switch to TypedVar everywhere, add Be to MatchInfo * cleanup, commit to TypedVar * More cleanup * document bug, already fixed * clean up bug stuff / test case for termeq bug * small cleanup before merge * Remove unneeded todo * Restyle Exploration of the Lower Your Guards paper (#402) * Restyled by fourmolu * Restyled by hlint --------- Co-authored-by: Restyled.io <[email protected]> * Restyled by fourmolu (#403) Co-authored-by: Restyled.io <[email protected]> --------- Co-authored-by: restyled-io[bot] <32688539+restyled-io[bot]@users.noreply.github.com> Co-authored-by: Restyled.io <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Automated style fixes for #401, created by Restyled.
The following restylers made fixes:
To incorporate these changes, merge this Pull Request into the original. We
recommend using the Squash or Rebase strategies.
NOTE: As work continues on the original Pull Request, this process will
re-run and update (force-push) this Pull Request with updated style fixes as
necessary. If the style is fixed manually at any point (i.e. this process finds
no fixes to make), this Pull Request will be closed automatically.
Sorry if this was unexpected. To disable it, see our documentation.