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

Eyeglasses shape #13

Merged
merged 1 commit into from
Jun 23, 2022
Merged

Eyeglasses shape #13

merged 1 commit into from
Jun 23, 2022

Conversation

alperenkaran
Copy link
Contributor

Hello, and thank you for the great library!

Inspired by the paper “confidence sets for persistence diagrams” (https://arxiv.org/pdf/1303.7117.pdf), the eyeglasses shape replicates and generalizes the shape given in the paper (fig.8). You can use arbitrary radii, and specify the neck size.

Note: This is my very first pull request :)

@ctralie
Copy link
Member

ctralie commented Jun 13, 2022 via email

@alperenkaran
Copy link
Contributor Author

Hello @ctralie ! I was wondering if I have to do anything else after the pull request? I am excited about my first contribution to a project :)

@ctralie ctralie merged commit 120a960 into scikit-tda:master Jun 23, 2022
@ctralie
Copy link
Member

ctralie commented Jun 23, 2022

Thank you for reminding me to merge this, and for your contribution!

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.

None yet

2 participants