Ignite: Create notes about concepts in ignite

Created on 15 Mar 2018  路  12Comments  路  Source: pytorch/ignite

These should be put in a sphinx compatible format like what is done in pytorch

0.1.0 help wanted

Most helpful comment

ok I think this task is now done. I will open new ones for the metrics.rst, engines.rst and handlers.rst package references.

Once these are done I think we're ready to release 0.1

All 12 comments

@alykhantejani what do you think to write in these notes? Today we have empty files :

    concepts.rst
    engines.rst
    exceptions.rst 
    handlers.rst 
    index.rst 
    metrics.rst
    quickstart.rst 

In pytorch similar notes cover some particular notions or best practices.

I think we need to cover a few things:
1) An overview of the event system
2) Defining your own event handlers
3) An overview of the metrics API

These files were put in as a stub initially, but if you fancy making a start on this feel free to re-arrange/delete as you see fit.

cc @jasonkriss in case he had some thoughts around this

Agreed with @alykhantejani. The way I had envisioned this and stubbed it out is to have a "Concepts" note that has the 3 things @alykhantejani mentioned. And then a "Quickstart" guide that just gives a barebones run through of getting a project up and running. The other .rst files will be filled in with autodoc but we certainly may want to add some additional context to each one at the top before the actual autodoc parts.

That being said, this can definitely change if you wanna get started on it and prefer a different arrangement.

I see what you mean @alykhantejani @jasonkriss, I can start with "Quickstart".
What do we put in index.rst or it will be filled with autodoc ? The file engineS.rst probably become engine.rst ...

engines.rst should still be engines because of how we have structured the code. You shouldn't have to put anything in index.rst unless you really want to. You can just add stuff to quickstart.rst. That would be awesome!

@jasonkriss you can check the first version of quickstart here ;-)

@vfdev-5 That looks like a great start!

I think this is done now @vfdev-5 @jasonkriss? Let me know if you think this should still stay open

@alykhantejani following your comment there is missing:

  1. An overview of the metrics API

But I do not know where to put it : concepts.rst or metrics.rst (however metrics is at Package Reference)

@vfdev-5 I think concepts.rst is a good place for high-level overview of how to to use metrics but i think metrics.rst is a good place to explain the Metric API and what methods need to be implemented to create your own metrics.

ok I think this task is now done. I will open new ones for the metrics.rst, engines.rst and handlers.rst package references.

Once these are done I think we're ready to release 0.1

@alykhantejani and exceptions.rst too or not ?
As there is a file ...

Was this page helpful?
0 / 5 - 0 ratings

Related issues

Sudy picture Sudy  路  4Comments

karfly picture karfly  路  4Comments

TheCodez picture TheCodez  路  3Comments

vfdev-5 picture vfdev-5  路  3Comments

alykhantejani picture alykhantejani  路  3Comments