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

Add infinite earring #795

Merged
merged 4 commits into from
Oct 11, 2024
Merged

Add infinite earring #795

merged 4 commits into from
Oct 11, 2024

Conversation

StevenClontz
Copy link
Member

Addresses #792.

@prabau
Copy link
Collaborator

prabau commented Oct 11, 2024

Are you satisfied with the look on the web page (gigantic equation all by itself)? Also, we could be slightly more descriptive with something like:
"The subspace of ... defined by $X=...$, that is, the union of the circles centered at $(\frac 1n,0)$ of radius $\frac 1n$ for $n=1,2,\dots$."

Approving and leaving it up to you if you want to make some changes.

@StevenClontz
Copy link
Member Author

we could be slightly more descriptive with something like:

I like this suggestion, thanks. And thanks for the proactive approval. Editing and merging.

@StevenClontz StevenClontz merged commit 6489c95 into main Oct 11, 2024
1 check passed
@StevenClontz StevenClontz deleted the StevenClontz/infinite-earring branch October 11, 2024 03:25
@Jianing-Song
Copy link
Collaborator

@StevenClontz Add also that infinite earring is arc-connected and locally arc-connected, like S139?

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.

3 participants