Skip to content

DOC: note about query strings in HDFStore selection #6905

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 1 commit into from
Apr 17, 2014
Merged

DOC: note about query strings in HDFStore selection #6905

merged 1 commit into from
Apr 17, 2014

Conversation

cpcloud
Copy link
Member

@cpcloud cpcloud commented Apr 17, 2014

closes #6901

@cpcloud cpcloud added this to the 0.14.0 milestone Apr 17, 2014
@cpcloud cpcloud self-assigned this Apr 17, 2014
@cpcloud
Copy link
Member Author

cpcloud commented Apr 17, 2014

See #6901. That was the reported error, so I wanted to make that example explicit. It will fail with a syntax error if you interpolate the string into a query

@cpcloud
Copy link
Member Author

cpcloud commented Apr 17, 2014

I'll point out the single and double quote

@jreback
Copy link
Contributor

jreback commented Apr 17, 2014

maybe show that u can pass a string if u surround it with quotes

@cpcloud
Copy link
Member Author

cpcloud commented Apr 17, 2014

done

@cpcloud
Copy link
Member Author

cpcloud commented Apr 17, 2014

turning out to be more controversial than i would've expected

@jreback
Copy link
Contributor

jreback commented Apr 17, 2014

the locals interpolation is a bit magical (but useful)

I always use direct quotes -

@cpcloud
Copy link
Member Author

cpcloud commented Apr 17, 2014

speaking of that, i must've overlooked this when refactoring local vars API but shouldn't pytables follow that convention?

@jreback
Copy link
Contributor

jreback commented Apr 17, 2014

it should follow the same convention
problem?

@cpcloud
Copy link
Member Author

cpcloud commented Apr 17, 2014

i meant with the @ syntax. ... that doesn't seem to work

@cpcloud
Copy link
Member Author

cpcloud commented Apr 17, 2014

in the example from #6901

@jreback
Copy link
Contributor

jreback commented Apr 17, 2014

hmm
ok so is this more like eval or query?
it doesn't have context so maybe should be more like eval in that locals need @ ?

@cpcloud
Copy link
Member Author

cpcloud commented Apr 17, 2014

i'll make a new issue, merge this doc PR?

@jreback
Copy link
Contributor

jreback commented Apr 17, 2014

yep

cpcloud added a commit that referenced this pull request Apr 17, 2014
DOC: note about query strings in HDFStore selection
@cpcloud cpcloud merged commit 39b9f79 into pandas-dev:master Apr 17, 2014
@cpcloud cpcloud deleted the query-doc branch April 17, 2014 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Docs IO HDF5 read_hdf, HDFStore
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SyntaxError on HDF queries where right-hand contains string delimiter
2 participants