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

API: timeseries accessors naming convention #9606

Closed
jreback opened this issue Mar 6, 2015 · 2 comments · Fixed by #37390
Closed

API: timeseries accessors naming convention #9606

jreback opened this issue Mar 6, 2015 · 2 comments · Fixed by #37390
Assignees
Labels
API - Consistency Internal Consistency of API/Behavior API Design Datetime Datetime data dtype good first issue
Milestone

Comments

@jreback
Copy link
Contributor

jreback commented Mar 6, 2015

we have a slightly odd naming convention for several index (and .dt) accessors

dayofweek
dayofyear

(and adding daysinmonth in #9605)

so these should prob be day_of_week

we should be able to maintain compat on these and just add the new ones.

@Violet-XiaoWeiHuang
Copy link

Hello @jreback ,
I'm a new comer to Pandas, I notice there're other timeseries accessors like 'weekofyear'. Do you want me to work on it?
Thank you.

@Violet-XiaoWeiHuang Violet-XiaoWeiHuang mentioned this issue Apr 15, 2019
4 tasks
@jbrockmendel jbrockmendel added the API - Consistency Internal Consistency of API/Behavior label Sep 20, 2020
@abhishekmangla
Copy link
Contributor

take

abhishekmangla added a commit to abhishekmangla/pandas that referenced this issue Oct 24, 2020
Rename dayofweek properties in Period, Datetime, and PeriodArray classes.
Add day_of_week property to respective tests for each class.
abhishekmangla added a commit to abhishekmangla/pandas that referenced this issue Oct 25, 2020
Similar to changes for day_of_week, create aliases for day_of_year and add to respective test files to ensure back-compat.
Also fix whatsnew rst file
abhishekmangla added a commit to abhishekmangla/pandas that referenced this issue Oct 26, 2020
Rename dayofweek properties in Period, Datetime, and PeriodArray classes.
Add day_of_week property to respective tests for each class.
abhishekmangla added a commit to abhishekmangla/pandas that referenced this issue Oct 26, 2020
Similar to changes for day_of_week, create aliases for day_of_year and add to respective test files to ensure back-compat.
Also fix whatsnew rst file
@jreback jreback modified the milestones: Contributions Welcome, 1.2 Oct 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API - Consistency Internal Consistency of API/Behavior API Design Datetime Datetime data dtype good first issue
Projects
None yet
5 participants