Skip to content

Add DatetimeIndex.day_of_year Method #1723

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

Closed
dhirschfeld opened this issue Aug 2, 2012 · 1 comment
Closed

Add DatetimeIndex.day_of_year Method #1723

dhirschfeld opened this issue Aug 2, 2012 · 1 comment
Labels
Refactor Internal refactoring of code
Milestone

Comments

@dhirschfeld
Copy link
Contributor

PeriodIndex has a day_of_year method, but DatetimeIndex doesn't. This makes it hard to write generic code which can take either.

Given they both have dayofyear attributes it may be best to remove the day_of_year method so that there is only one obvious way to do it.

@wesm wesm closed this as completed in 0fca796 Aug 9, 2012
@dhirschfeld
Copy link
Contributor Author

@wesm just wondering what the appropriate forum is to bring up issues which may or may not be bugs since I'm still learning the package. Is it best to bring them up on the ML first and let you create issues if required or should I open the issues directly (like I did this one) and let you triage them if they turn out to be... non-issues?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Refactor Internal refactoring of code
Projects
None yet
Development

No branches or pull requests

2 participants