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

Fix issue 109 hypothesis index 0 #124

Conversation

artiom-matvei
Copy link
Contributor

@artiom-matvei artiom-matvei commented Oct 7, 2024

105 fix needs to be merged first because of the csv files moving to data folder @vincentarelbundock

@artiom-matvei artiom-matvei force-pushed the fix_issue_109_hypothesis_index_0 branch from ff47a99 to 0bcbad7 Compare October 18, 2024 00:57
@artiom-matvei artiom-matvei force-pushed the fix_issue_109_hypothesis_index_0 branch from 0bcbad7 to 096aa99 Compare October 18, 2024 01:02
@artiom-matvei artiom-matvei marked this pull request as ready for review October 18, 2024 01:04
@vincentarelbundock
Copy link
Owner

I merged the other PR, but there now appears to be conflicts that should be fixed before this can be merged.

@vincentarelbundock
Copy link
Owner

This looks really great! I've very pleased with these changes. Thanks!

@vincentarelbundock vincentarelbundock merged commit 2853f95 into vincentarelbundock:main Oct 18, 2024
5 checks passed
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