Skip to content
This repository has been archived by the owner on Feb 26, 2021. It is now read-only.

Support compute normal form in Agda 2.6 #41

Closed
banacorn opened this issue Dec 27, 2016 · 5 comments
Closed

Support compute normal form in Agda 2.6 #41

banacorn opened this issue Dec 27, 2016 · 5 comments
Assignees
Labels

Comments

@banacorn
Copy link
Owner

The protocol has changed in Agda-2.6
https://github.com/banacorn/agda-mode/wiki/Conversations-between-Agda-&-agda-mode

@banacorn banacorn added the bug label Dec 27, 2016
@banacorn banacorn self-assigned this Dec 27, 2016
@banacorn banacorn mentioned this issue Dec 27, 2016
@ruhatch
Copy link

ruhatch commented Dec 27, 2016

So for normalisation outside a goal we now have

  | Cmd_compute_toplevel B.ComputeMode
                     String

And inside the goal we have

  | Cmd_compute         B.ComputeMode
                    InteractionId range String

Where ComputeMode is defined as

data ComputeMode = DefaultCompute | IgnoreAbstract | UseShowInstance

deriving (Read, Eq)

@banacorn
Copy link
Owner Author

Thanks for pointing the problem out.
I have tested sending this to Agda and it works in the version 2.6

IOTCM "/some/path/to/file.agda" None Indirect (Cmd_compute_toplevel DefaultCompute "some expression")

@banacorn
Copy link
Owner Author

I think these are the commands that correspond to ComputeMode. They are:

  • C-c C-n: Compute normal form DefaultCompute
  • C-u C-c C-n: Compute normal form, ignoring abstract IgnoreAbstract
  • C-u C-u C-c C-n: Compute and print normal form of show UseShowInstance

Prior to 2.6 there were only DefaultCompute and IgnoreAbstract so it was represented with a Bool.

@markokoleznik you may want to look at this, they are messing with the protocol again 😂.

banacorn added a commit that referenced this issue Dec 27, 2016
Prior to 2.6 there were only two modes of computation, now there are three.
Fix #39 and #41
@ruhatch
Copy link

ruhatch commented Dec 27, 2016

Thanks for fixing that up! Do you have an ETA for when this might hit stable release? Just wondering whether to install the dev version.

@banacorn banacorn changed the title Support Agda 2.6 Support compute normal form in Agda 2.6 Dec 27, 2016
@banacorn
Copy link
Owner Author

banacorn commented Dec 27, 2016

I haven't had the chance to do much testing (it's still mostly manual 😭), but I think it's good to go. What could possibly go wrong?? 😬 *microwaving popcorn*

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

No branches or pull requests

2 participants