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

Refactor Eckmann-Hilton #788

Merged
merged 29 commits into from
Sep 24, 2023
Merged

Conversation

morphismz
Copy link
Contributor

@morphismz morphismz commented Sep 19, 2023

This pr formalizes eckmann-hilton as the naturality condition of htpy-identification-left-whisk.

I added a file just for eckmann-hilton since it was over crowding the double loop space file. We will later need lemmas showing that, e.g., eckmann-hilton is preserved by tr and arbitrary functions, which now have a natural home in the new file. I wrote out an explanation preceding this new construction of eckmann-hilton. However, the explanation is long. If it is too long, let me know and maybe suggest a part that is unnecessary.

I added a higher-transport-along-identifications file because of cyclic module dependency issues. This will probably need to be organized a bit better.

This pr resolves a task from issue #702

@EgbertRijke
Copy link
Collaborator

EgbertRijke commented Sep 19, 2023

Very good! Instead of speaking of "higher transport", how about speaking of "transport along higher identifications"? This may seem like a minor detail, but then we can call the file transport-along-higher-identifications and it will appear right there among the other transport-along files.

@EgbertRijke
Copy link
Collaborator

EgbertRijke commented Sep 19, 2023

Would you also be so kind to add the new transport-along-higher-identifications file to the overview tables in foundation-core/identity-types and foundation/identity-types?

See here how it should look like: https://unimath.github.io/agda-unimath/foundation.identity-types.html#list-of-files-directly-related-to-identity-types

@fredrik-bakke fredrik-bakke changed the title refactor eckmann hilton Refactor Eckmann-Hilton Sep 19, 2023
@morphismz
Copy link
Contributor Author

Very good! Instead of speaking of "higher transport", how about speaking of "transport along higher identifications"? This may seem like a minor detail, but then we can call the file transport-along-higher-identifications and it will appear right there among the other transport-along files.

That's a much better, name! Will do later today

@morphismz
Copy link
Contributor Author

Would you also be so kind to add the new transport-along-higher-identifications file to the overview tables in foundation-core/identity-types and foundation/identity-types?

See here how it should look like: https://unimath.github.io/agda-unimath/foundation.identity-types.html#list-of-files-directly-related-to-identity-types

Should I also add the eckmann hilton file to this table? Not sure if it counts, since its specific to loops

@EgbertRijke
Copy link
Collaborator

There aren't any files specific about loop spaces in this table. Perhaps a smaller table in synthetic-homotopy-theory.loop-spaces that points to all the files about loops would be better.

@morphismz
Copy link
Contributor Author

There aren't any files specific about loop spaces in this table. Perhaps a smaller table in synthetic-homotopy-theory.loop-spaces that points to all the files about loops would be better.

I can go ahead an make that, though the only thing I can think of adding is the eckmann-hilton file. And that has more to do with double loop spaces than it does with spaces.

@EgbertRijke
Copy link
Collaborator

If we would make a table for anything to do with loop spaces of general pointed types, then it would make sense to include the double loop spaces and iterated loop spaces.

A quick search for the keyword "loop" on the indexing file for synthetic homotopy theory page

https://unimath.github.io/agda-unimath/synthetic-homotopy-theory.html

highlights all the files you could list in that table (9 in total, but 10 with your new EH file).

@morphismz morphismz marked this pull request as ready for review September 22, 2023 20:22
Copy link
Collaborator

@fredrik-bakke fredrik-bakke left a comment

Choose a reason for hiding this comment

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

I think it is really nice that you add context and discussion about the Eckmann-Hilton argument!

src/synthetic-homotopy-theory/eckmann-hilton.lagda.md Outdated Show resolved Hide resolved
src/synthetic-homotopy-theory/loop-spaces.lagda.md Outdated Show resolved Hide resolved
src/synthetic-homotopy-theory/eckmann-hilton.lagda.md Outdated Show resolved Hide resolved
src/synthetic-homotopy-theory/eckmann-hilton.lagda.md Outdated Show resolved Hide resolved
src/synthetic-homotopy-theory/eckmann-hilton.lagda.md Outdated Show resolved Hide resolved
@EgbertRijke
Copy link
Collaborator

EgbertRijke commented Sep 23, 2023

I had a quick look and your PR looks absolutely beautiful Raymond! Really good!

@morphismz
Copy link
Contributor Author

I had a quick look and your PR looks absolutely beautiful Raymond! Really good!

This might be the least number of formatting errors I've made in a pr 😎 We'll see if I can keep it up and turn it into a streak lmao

@EgbertRijke EgbertRijke merged commit b94ee59 into UniMath:master Sep 24, 2023
@morphismz morphismz deleted the refactor-eckmann-hilton branch September 25, 2023 03:14
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.

3 participants