Skip to content

Permutations and Renaming#13

Merged
EwenBC merged 6 commits intojespercockx:masterfrom
EwenBC:master
Jan 22, 2025
Merged

Permutations and Renaming#13
EwenBC merged 6 commits intojespercockx:masterfrom
EwenBC:master

Conversation

@EwenBC
Copy link
Collaborator

@EwenBC EwenBC commented Jan 20, 2025

No description provided.

	modified:   src/Scope/Renaming.agda
	modified:   src/Scope.agda

	modified:   src/Scope/Renaming.agda
	modified:   src/Scope.agda

 Date:      Tue Jan 21 09:56:14 2025 +0100
 Changes to be committed:
	modified:   src/Scope.agda
	modified:   src/Scope/Renaming.agda
 Untracked files:
	result

 Changes to be committed:
	modified:   src/Scope.agda
	modified:   src/Scope/Renaming.agda
 Changes to be committed:
	modified:   src/Scope/Renaming.agda

 Changes to be committed:
	modified:   src/Scope/Renaming.agda
 Changes to be ignored:
	modified:   src/Scope/Diff.agda
subRight : α ⋈ β ≡ γ → β ⊆ γ
subRight p = < splitComm p >
{-# COMPILE AGDA2HS subRight inline #-}
{-# COMPILE AGDA2HS subRight #-}
Copy link
Owner

Choose a reason for hiding this comment

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

Why remove the inline here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I don't know exactly why: the test complained about it and erasing it did the trick, so I didn't look further.

diffCase (inToSub xp) yp
(λ _ → < Zero ⟨ IsZero refl ⟩ >)
(λ yp' → let ⟨ _ ⟩ res = permutationToRenamingRev p < yp' > in < inThere res >)
{-# COMPILE AGDA2HS permutationToRenamingRev #-} No newline at end of file
Copy link
Owner

Choose a reason for hiding this comment

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

Please make sure to add a newline at the end of the file (tip: you can enable files.insertFinalNewline in VS Code to do it automatically).

→ Permutation (x ◃ α) β
{-# COMPILE AGDA2HS Permutation deriving Show #-}

{-
Copy link
Owner

Choose a reason for hiding this comment

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

Do we still need this version for anything?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

No, I'll erase it in the next commit.

@EwenBC EwenBC merged commit 1176300 into jespercockx:master Jan 22, 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