FluxML / IRTools.jl

Mike's Little Intermediate Representation
MIT License
111 stars 36 forks source link

update docs #32

Closed aviatesk closed 4 years ago

aviatesk commented 4 years ago

several updates:

MikeInnes commented 4 years ago

Thanks for this. I think the functions which have docs should generally be added as exports; we should avoid having IRTools.Inner.* references in the docs, since things should either be exported and documented or neither.

aviatesk commented 4 years ago

thanks for the feedback :)

I think the functions which have docs should generally be added as exports; we should avoid having IRTools.Inner.* references in the docs, since things should either be exported and documented or neither.

Okay, I can follow your direction. So how about exporting them ?

I also found

are exported but not included as in the reference.

MikeInnes commented 4 years ago

Thanks!