Tractables / LogicCircuits.jl

Logic Circuits from the Juice library
https://tractables.github.io/LogicCircuits.jl/dev/
Apache License 2.0
48 stars 4 forks source link

Docs + Examples #5

Open khosravipasha opened 4 years ago

khosravipasha commented 4 years ago
khosravipasha commented 4 years ago

In the docs, add links to other related libaries:

RenatoGeh commented 4 years ago

Not sure if relevant to this issue exactly, but one thing that might be helpful for end-users (it would be helpful to me at least) would be to explicitly state the return types of functions (as well as arguments). I often find myself having to look at the code to know exactly what do I have to pass as arguments/receive as returns. Ideally documentation should be more than enough for end-users. Plus the compiler probably could use some type hinting to speed things up. Not sure how you guys feel about this? I'm willing to do this boring refactoring and explicitly write argument and return types.

guyvdbroeck commented 4 years ago

I'm certainly in favor, especially in the documentation strings.

khosravipasha commented 4 years ago

Yeah this is very nice to have.

On the documentation side, I initially thought there would be an option in documenter.jl to generate the signitures automatically, but last time I checked seems to be no option for it. I guess easiest option for now is too add them manually in the comments.

I think we have return types set for most functions, it's just they are not being added to the docs by documenter.

We can slowly start adding them, for example any time we modify something we add this type of info to the doc string. I will check to see whats a consistent way to add them nicely in the doc string. I think you can just copy paste the function signitures and add it in the comments as code.