-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
Add micro- and millsecond period intervals. #2145
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
Comments
The work to do is mainly in |
One important detail that I've run into is that the |
In |
I'm really not a C expert, so you can make any refactorings you like as long as the test suite passes. You'll have to use 64-bit integers for micro- and nanoseconds since a whole day's worth won't fit in a 32-bit integer. If you look around for |
I want to submit a pull request for this issue. I've just (re)-read the pandas contributing documentation and I'm not sure I understand whether I need to rebase. I have an upstream branch that I'm constantly merging with my fork of pandas git master and that I'm also merging into the feature branch for this issue (I check for upstream changes at least once per day). In that case it would seem that I do not need to worry about rebasing, but I just want to make sure. |
Rebasing will ensure that your commits apply cleanly on top of upstream (at that moment at least), If your git-fu can handle it, then rebase on top of upstream and force push to replace the old commits |
So for future requests I should never merge pandas git master back into my feature branch and just work on the branch from whatever state master was in when I created the branch? |
here's the way I have setup
you may need to do
to update master: in theory your feature branch is now updated
you may also need to do as long as someone else is not working ON THE SAME things in master, this will work to update your feature branch to master and not pollute the history |
@cpcloud, yes. Rebase if you'd like to be neater, but never merge upstream into a branch You can think of rebasing as cherry picking all your commits one by one on top a new jreback's recipe is just fine. |
I went through almost every commit and rebased and squashed as many as I could, but if I look at what this looks like if I were to submit a pull request I still get a bunch of "Merge with upstream/master ..." commits. Is it okay to submit this, or is there a way to get only the commits that don't have that line in the title? I promise I'll never merge my branch with upstream again in any future pull requests. :| |
By all means, if have something to contribute to the porject but got tangled up with git , open If you used "rebase --interactive" you can just "drop" the merge commits, Both of these are similar in terms of the end result. If you're an emacs user, magit has wonderful support for interactive rebasing, I use it constantly. |
Continued in #2670 |
I'm totally willing to implement this myself, but I'm not exactly sure where to begin. I tried adding a microsecond period, but it seems like there are many low level changes required (plib.pyx, period.py (obviously), period.c, period.h, datetime.pyx, datetime.pxd and others that I can't remember off the top of my head).
Is there some documentation or a list of files and functions that need to be changed to add a period interval? If not, I can try and assemble one.
The text was updated successfully, but these errors were encountered: