-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
Simplify algebra in Week and SemiMonth offsets #18278
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
Simplify algebra in Week and SemiMonth offsets #18278
Conversation
Codecov Report
@@ Coverage Diff @@
## master #18278 +/- ##
==========================================
- Coverage 91.4% 91.38% -0.03%
==========================================
Files 164 164
Lines 49878 49852 -26
==========================================
- Hits 45590 45555 -35
- Misses 4288 4297 +9
Continue to review full report at Codecov.
|
do we have sufficient tests for all of these cases? |
As in #18263, I think so, yes. |
Looks like between the three active PRs unifying the Side-note. I went to check this on codecov by clicking the badge on pandas GH page and the link just goes to the image file for the badge itself. Looks like all the badges do that. I doubt that's the intended behavior. |
hmm thought that worked. |
thanks! |
you can click on the codecov report in the pr to see things FYI |
This PR is careful to only touch Week-base and SemiMonth-based offsets, i.e. has no overlap with #18263. It does, however, simplify the algebra such that the end result looks a lot like 18263.
git diff upstream/master -u -- "*.py" | flake8 --diff