panedr icon indicating copy to clipboard operation
panedr copied to clipboard

Add docs? (if needed)

Open IAlibay opened this issue 3 years ago • 4 comments

Adding this issue here as it might be a feature of the ongoing GSoC.

It might be worth consider what, if any, docs should be added / deployed for panedr.

IAlibay avatar Jun 11 '22 08:06 IAlibay

General addition of class docstrings would be a good starting point @BFedder - it'll let you double check that what you are making the same assumptions about how the classes are behaving as how @jbarnoud wrote them.

IAlibay avatar Jun 11 '22 08:06 IAlibay

The code tries to follow the gromacs code as much as possible. It is a good place to start to understand.

jbarnoud avatar Jun 11 '22 08:06 jbarnoud

There should be minimal docs — we tend to preach that any package should have

  • docs
  • tests at sufficient coverage
  • defined API

so we should also have them for panedr.

orbeckst avatar Jun 17 '22 22:06 orbeckst

There used to be a single exposed function and it was documented in the readme 😛

jbarnoud avatar Jun 17 '22 23:06 jbarnoud