-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
DOC: doc building deps #6059
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
Comments
+1 And maybe |
technically its still in the docs building...and technically pandas still supports it....am thinking we should drop in it 0.14 though...... |
@jorisvandenbossche so I have a virtual envirnment where I can do happens whether I build in-place or in the build dir .... sphinx 1.2 (or do I need newer?) |
OK, no problem (but +1 on deprecating/dropping it). And it's not anymore a dependency for the docs building thanks to y-p: #5921 |
For building the docs you will need or older (1.1.3) or newer (1.2.1, bug-fix release from a few days ago) |
Normally, if you do And once you have that, just doing |
that's all fine.....but the doc build is still picking up the wrong version (I also installed the built pandas in the virtual environ...looks like the correct version....) |
Ah, the docs are picking the wrong version. Well, I also had that issue some days ago after some upgrading/downgrading sphinx. For me, it had something to do with the sphinx-build script which referenced not the correct environment. I don't know if it is the same issue (for me it was with a conda environment and on Windows), but if so:
|
its odd the built docs @y-p ....seem to have the same issue: looking into it (see that iloc fails though it works correctly with ipython) |
Does this path look right to you @jorisvandenbossche
|
Ah, yes, I also saw that. But I think this is another issue, because I checked and I also have that in my latest local doc build (the |
yep...let me see if I can reproduce |
@jorisvandenbossche any way to run the doc-build and have it break when I put in |
Hmm, not directly an idea. The code is run in an embedded |
@jorisvandenbossche tried that....can't get it break into the shell....see if i can nail it down.... using print statements |
Also, it's not a typical error, as errors in the code example are printed out in the console where you type |
Didn't dd8be69 remove st from the deps for building the docs?? |
@y-p yes indeed #6059 (comment). The task bullet is then for removing it from show_versions. |
BLD: add IPython and sphinx to show_versions #6059
It's not hurting anyone, and I think it's still used by some tests. Why remove it? |
its fine....ok closing |
that's good |
Sometimes @jreback, I think you're two different people. and not just because |
hmm shall I take that as a compliment ? |
yes. I've already updated print_versions in #6064, is there anything else to do here? |
no that looks fine was just thinking we should remove scikit.timeseries from orint_versions as we don't have a dep for it for anything (docs or code) |
Like I mentioned earlier, still used as validation in tests:
|
didn't see that ok then |
so turns out that those I guess the obvious solution is the right one! (a bug and not a config issue) |
Can confirm the odd error in the docs in gone! @jreback Did you succeed to get the docs built with pandas master? |
@jorisvandenbossche yes error is gone...it was a 'real' error, pretty tricky to reproduce but squashed thanks for the help |
[ ] drop support scikit.timeseries in docs (and remove from print_versions.py)The text was updated successfully, but these errors were encountered: