Also, Sphinx has some troubles with autodocing some attributes and wrongly adds = None in the documentation. I propose to fix this problem upstream instead of working around by separating our documentation and code.
I don’t see any benefits. Also, waiting until the = None problem is fixed in Sphinx will take too long and since the extra work has already been done to fix this on our side, I don’t see the need to change everything again.
Again, the benefit of this change is the reduction documentation fragmentation. Code documentation (docstrings etc., not tutorials or examples) must be in one place and not spread between .py and .rst files. This is more important than any stylistic consideration in my opinion. Our extra work was a work-around and work-arounds are error prone and should be removed if there's a better solution (like in this pull request).