You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The SAE Lens API docs don't automatically list all code in the library (see here: https://jbloomaus.github.io/SAELens/api/). It would be good to make it so all functions / docstrings are present.
The text was updated successfully, but these errors were encountered:
Do you mean that you want all members (including those without docstrings) of the publicly-exposed classes/methods to show up on the API page of the documentation site?
If so, that can be done with a small edit to mkdocs.yml
However, it makes the API page really long, mostly because of the many class and instance attributes (they take up space not only in the page but also in the Table of Contents).
I've gone through the mkdocstrings/mkdocstrings-python documentation, and I can't find a way to include methods with no docstring but exclude attributes with no doc string.
Would you (the maintainers) be open to a PR that
a) also added these options to the same part of mkdocs.yml (in addition to the above)
which still leaves the page super long but avoids functions getting lost amidst the attributes in the Table of Contents
or b) skipped the approach described so far and just added docstrings to all classes/functions that're publicly-exposed by sae_lens/__init__.py's __all__ assignment (plus all non-'private' function members of those classes)?
If neither of those approaches would be satisfactory, does another approach come to mind that the maintainers would prefer?
Proposal
The SAE Lens API docs don't automatically list all code in the library (see here: https://jbloomaus.github.io/SAELens/api/). It would be good to make it so all functions / docstrings are present.
The text was updated successfully, but these errors were encountered: