pypeit / PypeIt-development-suite

8 stars 11 forks source link

README formatting #223

Closed kbwestfall closed 2 years ago

kbwestfall commented 2 years ago

Goes with https://github.com/pypeit/PypeIt/pull/1488. It just changes the formatting of the README file from markdown to rst. This allows me to include it verbatim in the pypeit docs.

badpandabear commented 2 years ago

It looks like the code block under the "If you’re using Google File Stream" is broken. The vs code viewer doesn't like it and it also looks wrong on readthedocs.

kbwestfall commented 2 years ago

It looks like the code block under the "If you’re using Google File Stream" is broken. The vs code viewer doesn't like it and it also looks wrong on readthedocs.

Good catch, @badpandabear ! I had an indentation error. I think I've fixed it now. Can you confirm? (It may take a little while for readthedocs to update, but you can check in vs code.)

debora-pe commented 2 years ago

Hi @kbwestfall I don't know if this is a problem with my machine, but I tried to run ./update_docs and also make html but I was not able to generate the docs and I got this error:

wget -O ./include/dev_suite_readme.rst https://raw.githubusercontent.com/pypeit/PypeIt-development-suite/rst_readme/README.rst
make[1]: wget: No such file or directory
make[1]: *** [apirst] Error 1
make: *** [html] Error 2
kbwestfall commented 2 years ago

Hi @kbwestfall I don't know if this is a problem with my machine, but I tried to run ./update_docs and also make html but I was not able to generate the docs and I got this error:

wget -O ./include/dev_suite_readme.rst https://raw.githubusercontent.com/pypeit/PypeIt-development-suite/rst_readme/README.rst
make[1]: wget: No such file or directory
make[1]: *** [apirst] Error 1
make: *** [html] Error 2

Hrmm... Do you get the same error when you copy the command into a terminal window? What happens if you add quotations around the url and/or the outptut file name?

badpandabear commented 2 years ago

Is it possible @debora-pe doesn't have wget installed?

debora-pe commented 2 years ago

@kbwestfall @badpandabear Dusty you are absolutely right! I had a problem with wget (not installed). Now it works! Thank you!