Skip to content
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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Update readme and contributing #590

wants to merge 6 commits into from

Conversation

f-brinkmann
Copy link
Member

@f-brinkmann f-brinkmann commented Apr 19, 2024

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:

  • Adapt README and CONTRIBUTING to new pyfar-wide help section
  • Flatten toc for Getting started and Contributing section in the docs to get a cleaner start page.
  • Check link to help section in README and the link to the community in Contributing after merging ttps://github.com/Harmonize documentation between pyfar packages gallery#20

update readme and contributing
@f-brinkmann f-brinkmann added the documentation Everything related to docstrings and comments label Apr 19, 2024
@github-actions github-actions bot added this to In progress in Code Backlog Apr 19, 2024
@ahms5 ahms5 added the 0.6.6 label Apr 19, 2024
Copy link
Member

@ahms5 ahms5 left a 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 Show resolved Hide resolved
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
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
.. _pyfar documentation: https://pyfar.readthedocs.io/en/stable
.. _pyfar documentation: https://pyfar.readthedocs.io

Copy link
Member Author

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
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
.. _SoundFile: https://pysoundfile.readthedocs.io/en/stable
.. _SoundFile: https://pysoundfile.readthedocs.io

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

CONTRIBUTING.rst Outdated
Comment on lines 72 to 75
Guide lines
-----------

Before you submit a pull request, check that it meets these guidelines:
You can find our general guidelines in the `pyfar community`_.
Copy link
Member

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.

Code Backlog automation moved this from In progress to Review in progress Apr 22, 2024
@f-brinkmann f-brinkmann requested a review from ahms5 April 22, 2024 11:29
Copy link
Member

@ahms5 ahms5 left a 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
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
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
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
.. _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
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
.. _help section: https://pyfar-gallery.readthedocs.io/en/latest/help.html
.. _help section: https://pyfar-gallery.readthedocs.io/en/latest/help

@f-brinkmann f-brinkmann requested a review from ahms5 April 22, 2024 13:14
Code Backlog automation moved this from Review in progress to Reviewer approved Apr 22, 2024
@ahms5 ahms5 added the cookiecutter Need to be updates in cookiecutter label Apr 25, 2024

.. _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
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
.. _pyfar example gallery: https://pyfar-gallery.readthedocs.io/en/latest/examples_gallery.html
.. _pyfar example gallery: https://pyfar.org

Comment on lines +23 to +27
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`_.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
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`_ .

Copy link
Member Author

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?

Code Backlog automation moved this from Reviewer approved to Review in progress May 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.6.6 cookiecutter Need to be updates in cookiecutter documentation Everything related to docstrings and comments
Projects
Code Backlog
Review in progress
Development

Successfully merging this pull request may close these issues.

None yet

3 participants