Skip to content

Feature Request: Expose more parsing options in html_read #13461

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.

Sign up for GitHub

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

Jump to bottom
Closed
gte620v opened this issue Jun 16, 2016 · 3 comments · Fixed by #13575
Closed

Feature Request: Expose more parsing options in html_read #13461

gte620v opened this issue Jun 16, 2016 · 3 comments · Fixed by #13575
Labels
Dtype Conversions Unexpected or buggy dtype conversions Enhancement IO HTML read_html, to_html, Styler.apply, Styler.applymap
Milestone
0.19.0

Comments

@gte620v
Copy link
Contributor

gte620v commented Jun 16, 2016

I am trying to parse some HTML and would love to have converters (https://github.com/pydata/pandas/blob/master/pandas/io/parsers.py#L1582-L1586) exposed to the read_html function.

Is there a reason why more parsing/all parsing options aren't exposed to TextParser at https://github.com/pydata/pandas/blob/master/pandas/io/html.py#L631-L634 ?

If not, would it be of interest to have a PR that adds this functionality?
@gte620v gte620v changed the title Expose more parsing options in html_read Feature Request: Expose more parsing options in html_read Jun 16, 2016
@jreback
Copy link
Contributor

jreback commented Jun 16, 2016

thought we had an issue about this, but closest is #10534 about passing dtypes thru. So this is quite similiar. It should be straightforward to implement.

@jreback jreback added Enhancement Difficulty Novice Dtype Conversions Unexpected or buggy dtype conversions IO HTML read_html, to_html, Styler.apply, Styler.applymap labels Jun 16, 2016
@jreback jreback added this to the Next Major Release milestone Jun 16, 2016
@gte620v
Copy link
Contributor Author

gte620v commented Jun 16, 2016

Cool. I'm happy to give it a shot if it is a feature that might get merged. Is it?

@jreback
Copy link
Contributor

jreback commented Jun 16, 2016

yes

gte620v added a commit to gte620v/pandas that referenced this issue Jul 7, 2016
@gte620v
Adding and arguments to for pandas-dev#13461 and pandas-dev#10534
afc7b2e
@gte620v gte620v mentioned this issue Jul 7, 2016
Merged
4 tasks
@jreback jreback modified the milestones: 0.19.0, Next Major Release Jul 20, 2016
jorisvandenbossche pushed a commit that referenced this issue Jul 21, 2016
@gte620v @jorisvandenbossche
ENH: Adding additional keywords to read_html for #13461 (#13575)
4d3b6c1
This was referenced Nov 7, 2016
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Dtype Conversions Unexpected or buggy dtype conversions Enhancement IO HTML read_html, to_html, Styler.apply, Styler.applymap
Projects
None yet
Milestone
0.19.0
Development

Successfully merging a pull request may close this issue.

ENH: Adding additional keywords to read_html for #13461 gte620v/pandas
2 participants
@jreback @gte620v