Conversation
|
How long does the |
It's actually surprisingly fast. Didn't extend the build time significantly in any noticeable way.
We may have to add it to every repo because each repo will have different sets of docs it needs to convert to markdown. |
|
One thing to keep in mind is token efficiency, i.e. try to cull useless info. For example, |
|
I've updated the script. |
|
Can you show the new output for some? |
|
I've updated the script to convert all the docs (except the user guide section). PR description also updated. |
maximlt
left a comment
There was a problem hiding this comment.
I have left a couple of questions.
Another one is that I checked how the markdown is made available on panel-material-ui. I went to https://panel-material-ui.holoviz.org/how_to/customize_palette.html and simply replaced .html with .md. It didn't work. Looking into the code I understand the URL needs to start with /markdown and indeed the file is available at https://panel-material-ui.holoviz.org/markdown/how_to/customize_palette.md. Do you know why the /markdown approach was chosen?
Could you please share a zip of the markdown folder built by this script? That would help review the changes.
We use a sphinx extension to set up redirects (e.g. when we remove/move a page, we redirect to a new one). I guess that's not supported here, but that it's also not so bad as LLMs can probably figure out where to find the new docs?
Is it planned to let people download directly the markdown file of a page, with a link/button from the page? It doesn't have to be done in this PR, I'm just asking if it's planned.
Will hvPlot's documentation be updated to let users know how they can use these markdown files and the holoviz skills? Again not expected to be done in this PR, I just want to know what is planned or not.
|
|
||
| [feature.doc.tasks] | ||
| docs-build-sphinx = 'sphinx-build -j auto -b html doc builtdocs' | ||
| docs-build-sphinx = 'sphinx-build -j 1 -d .sphinx-doctrees -b html doc builtdocs' |
There was a problem hiding this comment.
Isn't that making the docs build much slower?
There was a problem hiding this comment.
Why do we need to set -d .sphinx-doctrees?
There was a problem hiding this comment.
This was due to Sphinx parallel build issues. I first encountered the problem here: https://github.com/holoviz/holoviz/actions/runs/27723798712/job/82015403537
It's not much slower IMO.
There was a problem hiding this comment.
Ok but this is another repository. Does hvPlot suffer from the same error?
How much slower is it without being parallel? On which machine did it run?
There was a problem hiding this comment.
Ok but this is another repository. Does hvPlot suffer from the same error?
Yes. I think the problem is upstream from Sphinx
How much slower is it without being parallel? On which machine did it run?
I didn't do a comparison tbh but it ran on my mac pretty fast. These are basically just text files so it wasn't slow.
There was a problem hiding this comment.
I didn't do a comparison tbh but it ran on my mac pretty fast. These are basically just text files so it wasn't slow.
The Sphinx build runs all the notebooks, it is not just processing text files.
Yes. I think the problem is upstream from Sphinx
Ok so let's check whether the problem is still present upstream before disabling anything.
I'm not sure why but I imagine it was to keep the markdown docs separate from the html docs and to also make the link structures stable and easily findable.
We didn't really move a page here. We just made new ones. I'm not sure I understand the question here...
It wasn't originally planned but it sounds like a good idea we can probably do as well, maybe in a new PR.
Yes, this is planned. |
It's easily findable if
This is about the future. Imagine this scenario:
I'm fine with that too but want to make sure we understand the consequences of what we do.
For sure in another PR, and very likely in nbsite (I don't want us to start copy/pasting code from a repo to another, that's a maintenance nightmare).
Cool!! |
I don't think it's the most popular convention but I prefer this one tbh.
OK, in that case, it's not a problem because the via the pixi config, the markdown docs are rebuilt on every docs build. So if a file name is changed, the markdown file name will also be changed automatically. An LLM only needs to know the current file name and then it easily finds the corresponding |
|
I do like the idea of just replacing |
What I am asking is if it is a convention at all or something we came up with?
Let's ignore redirects for now (which are not supported but I think it'd not too bad as I care that we set up redirects for humans, less so for bots). Can you explain how an agent is supposed to find out about these markdown pages? Does it need to read the I haven't followed the work you all have done in this area so sorry for all the questions but I'd like to understand what we are setting up.
Not sure that would work as the extension might check for the existence of the new file, and we do the html->md conversion in a post-build step and not part of the Sphinx machinery. |
Ok good to know. Which other sites/projects use this convention?
Makes sense. |
:) pmui. This might be a question for @philippjfr I'd like to have this concluded sooner than later and we can apply it across all our repos, HoloViews in particular. |



Fixes #1730
Description
This PR adds a build script for generating
llms.txtand the relevant markdown docs.llms.txtfrom thedoc/directory:doc/user_guide/since it will be removed in the future.mdfiles and convert in-scope notebooks to markdown.github/workflows/docs.yamlbuild stepUnrelated
Streamingfromdoc/user_guide/index.mdto clear the doc build warnings.AI Disclosure
Tool & Model: GPT-5.3 Codex
Usage:
Screenshot of generated
llm.txtpage