braverock / blotter

blotter provides transaction infrastructure for defining transactions, portfolios and accounts for trading systems and simulation. Provides portfolio support for multi-asset class and multi-currency portfolios. Actively maintained and developed.
114 stars 50 forks source link

Add GSoC project related vignette chunks #100

Closed vi-to closed 5 years ago

vi-to commented 5 years ago

The aim is to add a vignette like documentation of the coding done during the GSoC.

All the code written is documented in place. However, the object documentation added during the last months is perhaps too verbose and could be overwhelming in the help window. Therefore I would suggest that once a complete vignette is in place and freely accessible we may revise internal docs so to provide main concepts and usages, while pointing to the vignette for more elaborated ones.

Also, there was a small portion of pre-existent text added by @jaymon0703 . I have tried not to mess with his intentions, meaning that I have not radically changed the structure of the document.

There is work in progress to provide an organic view on all the topics involved. It is expected that we will go through revisions to guarantee fluid explanations and practical examples, even after the GSoC.

braverock commented 5 years ago

Function documentation should always be complete without referring to a vignette. opening a pdf or html vignette isn't possible from a terminal, for instance, but ? is always available.

Vignettes are good for examples, tutorials, and more narrative explanation than is reasonable in documentation. vignettes also more readily incorporate graphics and tablular output. math is rendered completely in vignettes in ways which it is not in terminal documentation as well.

All of this is great, but we absolutely should not cripple the main function documentation in favor of "moving" things to vignettes.

vi-to commented 5 years ago

Thank you Brian for clear motivations. I agree with your view to keep a rich documentation there as well, it may be just worth doing some slight adjustments in the future. Time will tell, I am not doing any such change there for now.