-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
julia.readthedocs.org/en/latest search result #1382
Comments
Not meant to push here but just for your information if you'd like to track it down. The search result was good this morning (~7-8 hours ago) and it is showing python functions now. |
The original issue here was more obvious, it was a failure on our end during the requirements processing stage. However, that issue is no longer present, and the build servers are all running the same environment now. If we could track this down to a commit or build, that would help. It's showing Julia functions now that I've had a chance to look at it. If you notice constructs labeled as Python in the search, make note of the commit in |
@agjohnson I'll do that. It's sth like the following right? (note that this is a good commit) READTHEDOCS_DATA.commit
"cd8be5841e5f17aecd47409bc3962d458425b226" |
Indeed, that way we can pin the issue to a single build, which hopefully surfaces some differences in build output
Anthony Johnson |
READTHEDOCS_DATA.commit
"126b5d41947b96fe1db4a1450fc5c5d4beae5187" Is bad at this moment. |
It was good for a while today but it is now bad again. Current commit READTHEDOCS_DATA.commit
"2cbb72a0a813c367fd8b03d8d10273d0a429f015" |
READTHEDOCS_DATA.commit
"2c1ac1814f2b41824f69d827ccc6fc20ea4904aa" Still bad |
READTHEDOCS_DATA.commit
"59a1e9cac1ded47b08059bff31a4337175cc7a49" is bad again... |
Great, this is a good sample set, I'll poke around the associated builds and try to pinpoint where and why this is happening |
@agjohnson Thanks for looking into this! I might be imagining but it seems less likely to happen around noon EST (i.e. around this time everyday). |
I just see READTHEDOCS_DATA.commit
"e3dfa5678bd801a2f704f491db57c4416856a846" FWIW, this might be the first time I see a broken one around this time of the day recently.... |
Looked into this a little more, it's happening across all the servers, so there is no correlation between a single server here. I'll keep looking. There are some sphinx errors on overlapping object names, but normally that wouldn't cause anything like what you're seeing |
FWIW, I just noticed that functions are not the only thing that goes wrong. I was searching READTHEDOCS_DATA.commit
"a8e9189176b5f2c592ce576d867c75b1a75eb5cb" |
Ping. Any update on this recently? We've released the 0.4 version and now more and more people are confused by this now. (See the ping back from our issue tracker above) |
Sorry, this fell off my radar. I'll see if I can have any luck reproducing this inside/outside an RTD environment. This has been entirely elusive so far, so if you come up with any more information that could help track this down, that would be helpful. Something is triggering this on the Sphinx level, though I can't think of any reason triggering a build from RTD would have any different effect. |
Just did a bit more digging, and it looks like the functions are getting indexed as both Python and Julia objects. You can see this more clearly in a console:
So I'm guessing that there is some non-deterministic sorting going on with the search indexes, and when they are sorted in the first way, the "Julia" function comes up first, and the second way "Python" comes up. We need to look a bit more as why they're being indexed into both object domains. |
Yes, and I think this points to the need for a proper Julia domain. Overriding the Python name and prefix isn't enough to create proper indexes. All of the Julia objects are still stored in the domain representation as Python objects. Where the python domain is still loaded, this would cause duplication on indexed objects. For instance: https://github.com/sphinx-doc/sphinx/blob/master/sphinx/domains/python.py#L247-L255 |
I believe this code is where it is iterating over both domains, and finding the objects: https://github.com/sphinx-doc/sphinx/blob/master/sphinx/search/__init__.py#L278 -- I'm guessing our build servers have different dict sorting order, and so that is why sometimes it sorts properly and other times it sorts a different way. |
Closing this here and moving to JuliaLang/JuliaDoc/issues#16. A proper domain should resolve the issues with indexing. |
@agjohnson Thanks for looking into this. |
@yuyichao Sure thing, keep us posted if the domain doesn't fully resolve your issues. |
Ref JuliaLang/julia#11527
The search result in the latest julia doc returns "Python function" instead of
Julia function
from time to time without any change from julia side.The text was updated successfully, but these errors were encountered: