doc-fix: Fix for text outside of the dropdown and a grammar fix #30116
Conversation
|
I see multiple CI jobs failing. Is there a linting command that I should be using? |
|
The doc failure probably comes from a warning turned into error during the build, from build log: Maybe you need to align vertically your bullet point 2. content? You can draw some inspiration from the "Spectral Embedding" section which has a similar dropdown. You can also build the documentation locally, see doc for more details, and check the rendering. |
|
Made some fixes. Looks like the CI is stuck or disappeared? |
doc/modules/manifold.rst
Outdated
| :math:`O[D N k^3] + O[N d^6]`. The first term reflects a similar | ||
| cost to that of standard LLE. The second term comes from a QR | ||
| decomposition of the local hessian estimator. |
There was a problem hiding this comment.
doc CI is still red as far as I can see, maybe try this (one more space so that **Weight Matrix aligns vertically with the following lines that belong to the same bullet point)?
| :math:`O[D N k^3] + O[N d^6]`. The first term reflects a similar | |
| cost to that of standard LLE. The second term comes from a QR | |
| decomposition of the local hessian estimator. | |
| :math:`O[D N k^3] + O[N d^6]`. The first term reflects a similar | |
| cost to that of standard LLE. The second term comes from a QR | |
| decomposition of the local hessian estimator. |
There was a problem hiding this comment.
The CI was showing me a big circle for a very long time. And most of the icons were blurred for some reason. Made fixes.
There was a problem hiding this comment.
Thanks, the doc CI is happy now and the rendering looks good, see generated doc.
I have set auto-merge.
What does this implement/fix? Explain your changes.
Fixes the
Complexitydropdown insideHessian Eigenmappingsection insideManifold Learningmodule insideUnsupervised Learningchapter.Currently the dropdown inside the hessian
2.2.5. Hessian Eigenmappingdoesn't have the text as attached below.https://scikit-learn.org/stable/modules/manifold.html#hessian-eigenmapping
Added a grammar fix as well.