Skip to content

Conversation

luca-s
Copy link
Contributor

@luca-s luca-s commented Nov 15, 2016

@jreback
Copy link
Contributor

jreback commented Nov 15, 2016

tests pls

@jreback
Copy link
Contributor

jreback commented Nov 15, 2016

pls add an entry in whatsnew for 0.19.2

@jreback jreback added Bug Groupby Reshaping Concat, Merge/Join, Stack/Unstack, Explode labels Nov 15, 2016
@codecov-io
Copy link

codecov-io commented Nov 16, 2016

Current coverage is 85.28% (diff: 100%)

Merging #14663 into master will not change coverage

@@             master     #14663   diff @@
==========================================
  Files           140        140          
  Lines         50693      50693          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits          43234      43234          
  Misses         7459       7459          
  Partials          0          0          

Powered by Codecov. Last update 3f523f3...8db26db

@luca-s
Copy link
Contributor Author

luca-s commented Nov 16, 2016

Please let me know if that's ok because I wasn't sure where to add the test case.


- compat with ``dateutil==2.6.0`` for testing (:issue:`14621`)
- allow ``nanoseconds`` in ``Timestamp.replace`` kwargs (:issue:`14621`)
- fix ``pandas.cut`` with one negative value and one bin (:issue:`14652`)
Copy link
Contributor

@jreback jreback Nov 16, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug in pd.cut


expected = pd.Series([0, 0])

s = pd.Series([9., 9.])
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pandas\tools\tests\test_tile.py

lots of tests there, just add this one on., no need to create a new class. add a comment with the issue number

@luca-s
Copy link
Contributor Author

luca-s commented Nov 17, 2016

Thanks @jreback, now it should be fine.

@jreback jreback added this to the 0.19.2 milestone Nov 17, 2016
@jreback
Copy link
Contributor

jreback commented Nov 17, 2016

thanks!

@jreback jreback closed this in 2fc0c68 Nov 17, 2016
@luca-s luca-s deleted the issue_14652 branch November 17, 2016 13:33
@luca-s luca-s restored the issue_14652 branch November 17, 2016 13:34
jorisvandenbossche pushed a commit to jorisvandenbossche/pandas that referenced this pull request Dec 14, 2016
…s-dev#14652

closes pandas-dev#14652

Author: Luca Scarabello <[email protected]>
Author: Luca <[email protected]>

Closes pandas-dev#14663 from luca-s/issue_14652 and squashes the following commits:

8db26db [Luca Scarabello] Moved new test location to pandas\tools\tests\test_tile.py
90dd07d [Luca Scarabello] Updated whatsnew
d6b3da8 [Luca Scarabello] fixed flake8 compliance
fdc55b9 [Luca Scarabello] Added test case for pandas-dev#14652
d5790e2 [Luca Scarabello] updated whatsnew v0.19.2
2db0c7a [Luca] BUG: pandas.cut and negative values pandas-dev#14652

(cherry picked from commit 2fc0c68)
@luca-s luca-s deleted the issue_14652 branch February 17, 2017 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Groupby Reshaping Concat, Merge/Join, Stack/Unstack, Explode

Projects

None yet

Development

Successfully merging this pull request may close these issues.

BUG: pandas.cut and negative values

3 participants