New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update readme and contributing #590
base: main
Are you sure you want to change the base?
Conversation
update readme and contributing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, just a few comments!
README.rst
Outdated
.. _contribution guidelines: https://github.com/pyfar/pyfar/blob/develop/CONTRIBUTING.rst | ||
.. _pyfar workshop: https://mybinder.org/v2/gh/pyfar/gallery/main?labpath=docs/gallery/interactive/pyfar_introduction.ipynb | ||
.. _pyfar example gallery: https://pyfar-gallery.readthedocs.io/en/latest/examples_gallery.html | ||
.. _pyfar documentation: https://pyfar.readthedocs.io/en/stable |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.. _pyfar documentation: https://pyfar.readthedocs.io/en/stable | |
.. _pyfar documentation: https://pyfar.readthedocs.io |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
README.rst
Outdated
.. _pyfar.org: https://pyfar.org | ||
.. _read the docs: https://pyfar.readthedocs.io/en/latest | ||
.. _SoundFile: https://pysoundfile.readthedocs.io/en/latest/ | ||
.. _SoundFile: https://pysoundfile.readthedocs.io/en/stable |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.. _SoundFile: https://pysoundfile.readthedocs.io/en/stable | |
.. _SoundFile: https://pysoundfile.readthedocs.io |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
CONTRIBUTING.rst
Outdated
Guide lines | ||
----------- | ||
|
||
Before you submit a pull request, check that it meets these guidelines: | ||
You can find our general guidelines in the `pyfar community`_. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe we can add a general intro at the beginning of this doc. something that explains the general and specific guidelines. Maybe this part can be moved to the beginning so it is clear from the start.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! small comments, then I will approve.
CONTRIBUTING.rst
Outdated
helps, and credit will always be given. | ||
helps, and credit will always be given. The following helps you to start | ||
contributing to pyfar. General guidelines for example regarding the style | ||
of code and documentation are part of the You can find our general |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
of code and documentation are part of the You can find our general | |
of code and documentation are part of our general |
CONTRIBUTING.rst
Outdated
- check gallery in `binder`_ | ||
|
||
.. _binder: https://mybinder.org/v2/gh/pyfar/gallery/main?filepath=docs/gallery | ||
.. _pyfar community: https://pyfar-gallery.readthedocs.io/en/latest/community.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.. _pyfar community: https://pyfar-gallery.readthedocs.io/en/latest/community.html | |
.. _pyfar community: https://pyfar-gallery.readthedocs.io/en/latest/community |
the old link wont work.
README.rst
Outdated
.. _libsndfile: http://www.mega-nerd.com/libsndfile/ | ||
.. _help section: https://pyfar-gallery.readthedocs.io/en/latest/help.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.. _help section: https://pyfar-gallery.readthedocs.io/en/latest/help.html | |
.. _help section: https://pyfar-gallery.readthedocs.io/en/latest/help |
|
||
.. _contribution guidelines: https://github.com/pyfar/pyfar/blob/develop/CONTRIBUTING.rst | ||
.. _pyfar workshop: https://mybinder.org/v2/gh/pyfar/gallery/main?labpath=docs/gallery/interactive/pyfar_introduction.ipynb | ||
.. _pyfar example gallery: https://pyfar-gallery.readthedocs.io/en/latest/examples_gallery.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.. _pyfar example gallery: https://pyfar-gallery.readthedocs.io/en/latest/examples_gallery.html | |
.. _pyfar example gallery: https://pyfar.org |
The `pyfar workshop`_ gives an overview of the most important pyfar | ||
functionality and is a good starting point. It is part of the | ||
`pyfar example gallery`_ that also contains more specific and in-depth | ||
examples. The `pyfar documentation`_ gives a detailed and complete | ||
overview of pyfar. All these information are available from `pyfar.org`_. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The `pyfar workshop`_ gives an overview of the most important pyfar | |
functionality and is a good starting point. It is part of the | |
`pyfar example gallery`_ that also contains more specific and in-depth | |
examples. The `pyfar documentation`_ gives a detailed and complete | |
overview of pyfar. All these information are available from `pyfar.org`_. | |
For a list of application examples including introductions to most important functionality check out the `pyfar example gallery`_ and `pyfar workshop`_ . All gallery example notebooks can be executed interactively without a local installation by clicking the mybinder.org button on the respective example. | |
A full overview of all features is found in the `pyfar documentation`_ . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I actually prefer the current version. It first links the workshop, which might be the easiest entry to the entrire package and explicitly links the gallery, not only pyfar.org. Would you be fine with leaving it as it is?
Which issue(s) are closed by this pull request?
Part of pyfar/gallery#20.
Changes in the readme might conflict with #574. I think the readme here might be the final version.
Changes proposed in this pull request: