Skip to content

[CLN] cy cleanup, de-duplication #21826

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

Merged
merged 4 commits into from
Jul 11, 2018
Merged

[CLN] cy cleanup, de-duplication #21826

merged 4 commits into from
Jul 11, 2018

Conversation

jbrockmendel
Copy link
Member

Small cleanups, remove some unnecessary casting, add typing in timezones.

@jreback
Copy link
Contributor

jreback commented Jul 9, 2018

looks fine. may want to run a perf check as touching some sensitive paths.

@jreback jreback added Datetime Datetime data dtype Clean labels Jul 9, 2018
@jbrockmendel
Copy link
Member Author

jbrockmendel commented Jul 9, 2018 via email

@codecov
Copy link

codecov bot commented Jul 9, 2018

Codecov Report

Merging #21826 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #21826   +/-   ##
=======================================
  Coverage    91.9%    91.9%           
=======================================
  Files         160      160           
  Lines       49892    49892           
=======================================
  Hits        45852    45852           
  Misses       4040     4040
Flag Coverage Δ
#multiple 90.28% <ø> (ø) ⬆️
#single 42.1% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5380fcd...13c4e8a. Read the comment docs.

@jreback jreback added this to the 0.24.0 milestone Jul 9, 2018
@jbrockmendel
Copy link
Member Author

jbrockmendel commented Jul 10, 2018

asv is raising git-related errors for me on both linux and mac. Is anyone else getting these?
update caused by me messing up the cloning, never mind.

@jbrockmendel
Copy link
Member Author

Results are probably meaningless, but at least they're all good-meaningless.

asv continuous -E virtualenv -f 1.1 master HEAD -b timeseries
[...]
    before     after       ratio
  [da6e26db] [13c4e8a0]
-    8.39ms     7.61ms      0.91  timeseries.ToDatetimeISO8601.time_iso8601_nosep
-    2.36ms     2.11ms      0.89  timeseries.ResampleSeries.time_resample('datetime', '1D', 'ohlc')
-    6.98μs     6.21μs      0.89  timeseries.DatetimeIndex.time_get('tz_aware')
-    5.91μs     5.01μs      0.85  timeseries.DatetimeIndex.time_get('tz_naive')

@jreback jreback merged commit 1d29dc4 into pandas-dev:master Jul 11, 2018
@jreback
Copy link
Contributor

jreback commented Jul 11, 2018

thanks !

@jbrockmendel jbrockmendel deleted the cnp2 branch July 11, 2018 01:20
Sup3rGeo pushed a commit to Sup3rGeo/pandas that referenced this pull request Oct 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Clean Datetime Datetime data dtype
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants