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

Eckmann-Hilton Updates #1133

Open
wants to merge 38 commits into
base: master
Choose a base branch
from
Open

Eckmann-Hilton Updates #1133

wants to merge 38 commits into from

Conversation

morphismz
Copy link
Contributor

Fixes links and makes consistent naming changes related to the Eckmann-Hilton file. Also adds an updated reference to the page.

Copy link
Collaborator

@VojtechStep VojtechStep 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 the update. I didn't yet go through the content in detail, so this is a preliminary review

src/foundation/homotopies.lagda.md Outdated Show resolved Hide resolved
@fredrik-bakke
Copy link
Collaborator

Hey Raymond, welcome back! It's good to see you're active again. Looking forward to seeing what's coming next. 🙂

@morphismz
Copy link
Contributor Author

Hey Raymond, welcome back! It's good to see you're active again. Looking forward to seeing what's coming next. 🙂

Hey Fredrik, thanks! I'm spinning up for the summer, hoping to get a good chunk of math formalized. Hoping to get the Hopf fibration finally all formalized. I figured I'd acquaint myself with the updates to the library by making a few small prs first. Though it seems like I should reacquaint myself with the style guide first 😄

@morphismz morphismz marked this pull request as draft April 28, 2024 22:53
@morphismz morphismz marked this pull request as ready for review April 28, 2024 23:25
@morphismz morphismz marked this pull request as draft May 23, 2024 02:14
@morphismz
Copy link
Contributor Author

morphismz commented May 23, 2024

I've added a bit more work to this pr. The main result is a finished computation of transporting along an eckmann-hilton identification, resolving one of the tasks in issue #702 .

There is a bit more clean up I need to do before this pr is ready for review. I'll mark it ready when the time comes.

@morphismz morphismz marked this pull request as ready for review May 26, 2024 18:36
@morphismz
Copy link
Contributor Author

morphismz commented May 26, 2024

I think this pr is now ready for review. There is one more lemma I will add later, namely computing transport along the eckmann-hilton 3-loop. But this relies on some infrastructure related to 3-automorphisms that I will be building in a different pr. So that lemma will wait.

Please let me know if I should tone down the prose, I feel like there is a lot on the Eckamnn-Hilton page. I added a bit of an explanation of how we compute transport along an Eckmann-Hilton identification. Let me know if there should be more or less of an explanation

Copy link
Collaborator

@VojtechStep VojtechStep left a comment

Choose a reason for hiding this comment

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

Great work! Those coherences look like a major PITA, so good job. I'd say 95% of my comments are just formatting. I think this amount of prose is very reasonable, it's always better to have more than less, and most of the content on that page is taken by arguments to lemmas which would be implicit in an ideal world. Keep it up!

src/foundation/commuting-squares-of-homotopies.lagda.md Outdated Show resolved Hide resolved
@morphismz
Copy link
Contributor Author

Those coherences look like a major PITA

Yes, yes they were...

Thanks for the diligent review process and all the feed back!

@morphismz
Copy link
Contributor Author

Okay, I tried messing around with the diagrams a bunch, but I couldn't get anything to look nice. I'll delete them for now, so this pr can get merged. I'll keep thinking about ways to explain those coherence (maybe linking to some external diagrams, idk). If I come up with something, I'll add that in another pr.

But, for now, I figured, I'll just try to get this pr merged.

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

Successfully merging this pull request may close these issues.

None yet

3 participants