-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
DOC: add doc for reading from DataFrame.to_html #3656
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
Conversation
ok to merge? |
just added more notes, so yep |
dont' use assert_frame_equal in the v0.11.1...instead print out the table |
perfect! fyi....since its ipython, the |
cool. i figured. pre-ipdb debugging habits die hard! |
hmm....can you rebase on top of master? |
Yeah sorry. Will do when I get to a machine... |
@jreback rebased. |
something failed...... |
yep saw it i'm on it, i'm getting a different error locally about recursion depth... |
prolly cause i changed the default of infer types...and thus everything is evil object arrays |
@jreback any changes to assert_frame_equal recently? not made by me... |
ugh why does |
I think @y-p changed something...didn't seem to affect me (btw you can NOT have it compare dtypes, there is an option) but I think you should have infer_dtypes=True by default, it makes sense |
ok...the example of reading in the output of |
yeh...dates are tough (though coerce will only convert if at least 1 item is a date)......(if all are not dates then it will leave it) |
wait but why is |
what is in the series? |
if u just do s = Series(['a']).convert_objects(convert_dates='coerce')
|
note new date conversion behavior jreback doc recommendations cannot pass a string to to_thml gah working notes modify tests infer_types back to true as per disc. with @jreback fix failing tests because of not correctly converted dates weird recursion error when using format spec instead of format strings
evil! I can see 'today' being parsed to today, but
|
@jreback travis is now all green |
ah yes when parsers become too powerful and take over the world! mwahaha! |
it's because |
and also the default of just returning the current time kicks in since the parse of |
kind of insane if u ask me... |
|
see #3671 |
ok...merging.....after my PR you can decide if you want to turn convert_dates back on (or even make it a separate option to |
DOC: add doc for reading from DataFrame.to_html
@cpcloud if u have a chance and I don't think any docs for to_html thxs |
@jreback btw should the |
maybe someone has and there's a reason for not supporting it...i haven't looked through issues for it |
as far as I know these methods should accept path_or_buf |
ok i will open a pr for any inconsistencies i find with that |
@jreback would u mind if i show the html output as rendered html in the examples? that way a user gets the final result and doesn't have to open up browser to see the rendered html... |
that would be nice |
hm how do u feel about a bunch of html files in a subdirectory of |
I think you can put what you need in |
as an example, I do this is io.rst (for showing how to read a legacy file)
|
No description provided.