Skip to content

BUG: #14095. Amend eval() resolvers kwarg to accept lists #14121

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
wants to merge 4 commits into from
Closed

BUG: #14095. Amend eval() resolvers kwarg to accept lists #14121

wants to merge 4 commits into from

Conversation

tom-bird
Copy link
Contributor

@codecov-io
Copy link

codecov-io commented Aug 30, 2016

Current coverage is 85.27% (diff: 100%)

Merging #14121 into master will increase coverage by <.01%

@@             master     #14121   diff @@
==========================================
  Files           139        139          
  Lines         50511      50537    +26   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits          43071      43097    +26   
  Misses         7440       7440          
  Partials          0          0          

Powered by Codecov. Last update 10bf721...227d734

@chris-b1
Copy link
Contributor

This should also be tested (and may need a fix) for pd.eval

@tom-bird
Copy link
Contributor Author

pd.eval seems ok taking a list or tuple

@jreback jreback added the Bug label Aug 31, 2016
@jreback
Copy link
Contributor

jreback commented Aug 31, 2016

@theultimatecrouton can you add a test for pd.eval as well. otherwise lgtm.

@tom-bird
Copy link
Contributor Author

Added

@jreback jreback added this to the 0.19.0 milestone Aug 31, 2016
@jreback jreback closed this in b2a73b8 Aug 31, 2016
@jreback
Copy link
Contributor

jreback commented Aug 31, 2016

thanks!

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

Successfully merging this pull request may close these issues.

BUG: resolvers in pandas.eval should accept list-like
4 participants