Docs
This is for all the documentation.
@flavio.tarsetti please see if the development installation guide fixes whatever issues you had!
Also please let me know of anything I should be adding to the dev guide.
-
Check to make sure the docs works with beat/docs
Merge request reports
Activity
- Resolved by Jaden DIEFENBAUGH
- Resolved by Jaden DIEFENBAUGH
added 3 commits
- 176aa73a - [docs] added line to install pyyaml to base conda env, #123 (closed)
- 276893f3 - [docs] added user guide via tutorial from slides
- 78e98968 - [docs] add editor breakdowns, #123 (closed)
@flavio.tarsetti could you check this out?
Also please check if this works with
beat/docs
since I don't know how that works!Note: There are some generic~fuzzy things in the docs now because stuff is going to be changing over the next few months, especially in the editors.
@jdiefenbaugh I will check this out and let you know
For
beat/docs
, you can clone the public packagegit@gitlab.idiap.ch:beat/docs.git
Then change (or not) the version line in before_build.sh script:
if [[ $pkg == "beat.editor" ]]; then version=docs fi
Right now I added docs (as your branch you are working on is named
docs
, soon of course it will generate the documentation from master)Then run the following:
./before_build.sh ../bob.admin/conda/conda-build.sh --python=3 conda
Where
./before_build.sh
will download all the packages in adoc/
folder and../bob.admin/conda/conda-build.sh --python=3 conda
will generate thesphinx/
folder with all the documentation.This way you will be able to check if the documentation you generated is fine before pushing a commit.
- Resolved by Jaden DIEFENBAUGH
- Resolved by Jaden DIEFENBAUGH
Could you also clarify to me (and also in the docs maybe) why we need the
/bin/beateditor --dev
to run this flask server ?I mean the complete beat.editor will actually work with
npm start
for thewebpack dev server
, so why two servers withflask
needed here?Edited by Flavio TARSETTIYou need
/bin/beateditor --dev
because the--dev
flag prevents the server from serving the distributable (prod) HTML/JS/CSS files. Since the HTML/JS/CSS will be served by the webpack dev server, you only need the REST API from the python server.The webpack dev server only serves the website stuff, not the API.
I'm a bit confused but I guess the answer to this question is maybe here:
https://gitlab.idiap.ch/beat/beat.editor/blob/docs/doc/api.rst#L7
But the doc is empty for this part.
@flavio.tarsetti anything else I need to change/etc.?
Thanks for the doc. It's good as it's really some kind of tutorial for the user guide, and it's very nice. Looks good to me.
Could you add the API documentation too as it's empty now.
It would be good to add some more stuff for the Development part in the JS stuff where you can describe a little more each modules as it's a bit light for now.
I think a good thing would be to start keeping all this up-to-date with the new modifications you are working on from now on. Maybe some kind of sphinx-js would be good too to keep the code documented as it's not documented for now.
Does this seem good ?
API Documentation
As far as the API docs go, shouldn't I actually be removing the API page completely? Correct me if I'm wrong, but the API documentation is only for packages providing an library/API for consumers. Since our package is purely an end-user app and so doesn't provide any library/API interface, shouldn't there not be an API page?
Dev docs
Sure I'll expand on that stuff, probably a good idea.
Sphinx-JS-like-thing
I wish there was a tool like Sphinx for modern JS :(. Unfortunately there's not yet. There's stuff like JSDoc, which is fine for oldschool JS but won't work for contemporary JS projects. There's also stuff like Storybook, which is super cool but not stable enough. I actually spent alot of time with both of these last year when I started this project, but either projects are abandoned, not mature~stable enough, or only support old JS. In awhile Storybook might end up being our best bet.
Keeping up-to-date
Yes agreed! It would be best to review the docs before an MR merge.