The difference being that the other pr automatically identifies collisions? Yes, this is simpler to read

Yes the other PR is automatic, while this one requires a user to add a function to custom_autosummary_names_with_new_suffix if there is a conflict. Since both solutions monkey patches, I think a simpler to read version is better.

There is an issue in sphinx regarding this issue: sphinx-doc/sphinx#1495

Yea, I wish this was less hacky. The logic of "check if file exist by not using the file system and renaming the file to something" is kind of weird to configure (from a sphinx user point of view)

This is ancient, but if it's not been fixed upstream in Sphinx (we should check), we should consider merging it before trying to create a zipped-HTML release of the docs.

We are still using this for update the docs for dash-docs.

Last time I looked at this, autosummary was using the filenames as the "source of truth" and this only works for case sensitive file systems.

From the original issue: sphinx-doc/sphinx#1495 it looks like this issue has been pushed up to "Some future version" milestone. (Sounds familiar eh?)

I have a feeling this would introduce new sphinx warnings and fail the CI (now that it catches sphinx warnings).

Edit - Oh wow it worked

Let's merge this one so that we can proceed on #17051?

This has broken the doc build:

Traceback (most recent call last):
  File "/home/circleci/miniconda/envs/testenv/lib/python3.8/site-packages/sphinx/cmd/build.py", line 276, in build_main
    app = Sphinx(args.sourcedir, args.confdir, args.outputdir,
  File "/home/circleci/miniconda/envs/testenv/lib/python3.8/site-packages/sphinx/application.py", line 244, in __init__
    self.setup_extension(extension)
  File "/home/circleci/miniconda/envs/testenv/lib/python3.8/site-packages/sphinx/application.py", line 398, in setup_extension
    self.registry.load_extension(self, extname)
  File "/home/circleci/miniconda/envs/testenv/lib/python3.8/site-packages/sphinx/registry.py", line 414, in load_extension
    metadata = setup(app)
  File "/home/circleci/project/doc/sphinxext/custom_autosummary_new_suffix.py", line 76, in setup
    for listener_id, obj in builder_inited_listeners.items():
AttributeError: 'list' object has no attribute 'items'

should I revert it or are you gonna do it?

@thomasjpfan - I was so happy to see you have a solution for this problem here, and sad to see it needed to be reverted.
Did you come up with something else instead?

@bsipocz The fix was added to sphinx directly in sphinx-doc/sphinx#7927 with the autosummary_filename_map feature. Scikit-learn makes uses the feature as follows: