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 anisotropic coefficient of variation #72

Draft
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

Teschl
Copy link
Contributor

@Teschl Teschl commented Oct 30, 2024

This adds the wrapper for the acv function to pytopotoolbox. I did not add a .ipynb file for the documentation for this function since there is not much to this function.

@Teschl
Copy link
Contributor Author

Teschl commented Oct 30, 2024

I changed the git tag to next week's release so it's failing right now. We can rerun the tests when the release has been created.

@Teschl Teschl marked this pull request as draft November 12, 2024 11:55
@Teschl
Copy link
Contributor Author

Teschl commented Nov 30, 2024

This contains the wrapper for the acv function and the new wrapping guide #76

@wkearn
Copy link
Contributor

wkearn commented Dec 4, 2024

Do you want to try to extract your function wrapping guide and make that a separate pull request? That would be very useful to have, and it seems like our progress on acv is stalled while we try to resolve its issues over in libtopotoolbox.

@Teschl
Copy link
Contributor Author

Teschl commented Dec 4, 2024

Yes that probably a good idea 👍

@Teschl Teschl mentioned this pull request Dec 4, 2024
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