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
feat: Implement fill() #113
feat: Implement fill() #113
Conversation
Codecov Report
@@ Coverage Diff @@
## master #113 +/- ##
==========================================
+ Coverage 93.53% 93.58% +0.05%
==========================================
Files 58 59 +1
Lines 1455 1482 +27
==========================================
+ Hits 1361 1387 +26
- Misses 94 95 +1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
I have today returned from holiday. I will try and look at this this evening. Thanks for the hard work. |
No problem, take your time (to be honest I forgot this PR) |
Small bump (I'm going through my open PRs to see if they can be merged/closed) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is really great work. Thanks a lot @etiennebacher and sorry for the slow response on this one.
No problem ;) |
Reproduce
tidyr::fill()
. I took the main part of the function (the filling algorithm) in a SO answer and I must say I'm not super confident of how it works, but it works and it's fast.A few tests are missing, for lists for example (see tidyr tests. There's also a weird message with groups (see last example).
Examples:
Created on 2022-08-26 by the reprex package (v2.0.1)