Extract custom sphinx stuff to package

[flying-sheep]

#100 did this:

• Remove scanpydoc dependency
• Add typed returns custom extension like in scanpy
• Use same autosummary class template as in scvi-tools (single page for classes with methods/attr tables)

why? shared code should live in a package. let’s revert that part and improve scanpydoc to enjoy shared goodness.

[adamgayoso]

I agree that scanpydoc could have these features (and renamed to scversedoc)

This could also include the custom linkcode functions to link source out to github #103

[grst]

Is it that bad to have a bit of vendored code, especially with template sync in place?

…

On Fri, 24 Feb 2023 at 19:59, Adam Gayoso ***@***.***> wrote: I agree that scanpydoc could have these features (and renamed to scversedoc) This could also include the custom linkcode functions to link source out to github #103 <#103> — Reply to this email directly, view it on GitHub <#113 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABVZRV73UDDUBVSXGCF6PILWZEAJ7ANCNFSM6AAAAAARM5FD3I> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[adamgayoso]

Is it that bad to have a bit of vendored code, especially with template
sync in place?

This is also a good point. I'm indifferent. Package makes it easier for people to use without using the template.

[flying-sheep]

I believe minimizing boilerplate and relying just on Git (for your project specific settings) and dependencies (for things maintained by other people) simplifies things. Sure, after learning Git and Python dependency management, some people also want to learn cruft and configure it to deal with boilerplate, but how many of our users are that?

What do you think? How realistic is it that our users will use template sync?

[Zethson]

What do you think? How realistic is it that our users will use template sync?

In practice, quite a few people get lazy with updating it. Nevertheless, these people also wouldn't update their scverse-doc dependencies.

But that shouldn't stop us from using any option