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: Fix usage of missing data type in univariate analysis #71

Merged
merged 2 commits into from
Sep 6, 2023

Conversation

mbelak-dtml
Copy link
Collaborator

No description provided.

@mbelak-dtml mbelak-dtml self-assigned this Aug 9, 2023
@mbelak-dtml mbelak-dtml force-pushed the fix/missing-data-type-univariate-analysis branch from 5575b45 to 045fef2 Compare August 9, 2023 14:26
@mbelak-dtml mbelak-dtml force-pushed the fix/missing-data-type-univariate-analysis branch from 045fef2 to 5903586 Compare August 14, 2023 09:34
@mbelak-dtml mbelak-dtml marked this pull request as ready for review August 14, 2023 09:36
lukany
lukany previously approved these changes Sep 6, 2023
@lukany
Copy link
Contributor

lukany commented Sep 6, 2023

@mbelak-dtml I believe the failed checked are caused by some transient issues. So it's IMHO safe to merge.

@mbelak-dtml
Copy link
Collaborator Author

mbelak-dtml commented Sep 6, 2023

@mbelak-dtml I believe the failed checked are caused by some transient issues. So it's IMHO safe to merge.

Interesting. The tests passed on the re-run. I don't see a reason why it would fail the first time. It retrieved the same poetry env from the cache on both occasions.

@mbelak-dtml mbelak-dtml added this pull request to the merge queue Sep 6, 2023
Merged via the queue into main with commit 7e9cd46 Sep 6, 2023
5 checks passed
@mbelak-dtml mbelak-dtml deleted the fix/missing-data-type-univariate-analysis branch September 6, 2023 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants