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

Handle numerical habits from HabitBull #1278

Merged
merged 1 commit into from
Feb 2, 2022

Conversation

hiqua
Copy link
Collaborator

@hiqua hiqua commented Jan 20, 2022

No description provided.

@hiqua hiqua force-pushed the fix_habit_bull_numerical branch 2 times, most recently from 9c9fb26 to b19f9ab Compare January 20, 2022 22:46
@hiqua hiqua marked this pull request as draft January 20, 2022 22:49
@hiqua hiqua marked this pull request as ready for review January 20, 2022 22:57
@hiqua hiqua requested a review from iSoron January 20, 2022 22:57
@hiqua hiqua removed the request for review from iSoron January 20, 2022 23:00
@hiqua hiqua marked this pull request as draft January 20, 2022 23:00
@iSoron
Copy link
Owner

iSoron commented Jan 28, 2022

Thank you for the PR, @hiqua. I will review it soon.

@iSoron iSoron merged commit 1d3bd48 into iSoron:dev Feb 2, 2022
@iSoron
Copy link
Owner

iSoron commented Feb 2, 2022

Everything looks good, @hiqua. A bit surprising to see that the CSV doesn't have anything to indicate that the habit is numeric, but I think your idea here of checking value > 1 is good. Thanks again!

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.

Import of HabitBull data assumes all habits are daily yes/no and mis-imports numeric value habit scores as No
2 participants