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

Improve docstrings for abstract hardware objects #1116

Merged

Conversation

fabcor-maxiv
Copy link
Contributor

General improvements to the docstrings of abstract hardware objects:

  • Add an Emits custom section for Sphinx's Napoleon extension
  • Fix docstrings formatting
  • Document some emitted signals
  • Move types from docstring to type hints when possible
  • Som formatting with black

GitHub: supersedes #866
GitHub: supersedes #947

@fabcor-maxiv fabcor-maxiv force-pushed the abstract-hwo-docstrings branch from f0791da to eba2843 Compare January 30, 2025 16:59
@marcus-oscarsson
Copy link
Member

Nice, good initiative !

@fabcor-maxiv fabcor-maxiv force-pushed the abstract-hwo-docstrings branch from eba2843 to 5ba1f68 Compare January 31, 2025 07:53
@fabcor-maxiv
Copy link
Contributor Author

Just now we are being hit by this deprecation: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/. :D

@marcus-oscarsson
Copy link
Member

Yeah I noticed as well, its not our week for documentation :)

@fabcor-maxiv fabcor-maxiv force-pushed the abstract-hwo-docstrings branch 2 times, most recently from 5090b37 to 4a7ccc3 Compare January 31, 2025 08:51
@fabcor-maxiv fabcor-maxiv marked this pull request as ready for review January 31, 2025 08:59
@fabcor-maxiv
Copy link
Contributor Author

Alright, so this is basically a rewrite of #866, plus some of the improvements I started on #947, and then some more improvements as I went. And it includes a black-formatting of AbstractCollect.py that was excluded up until now.

@marcus-oscarsson
Copy link
Member

marcus-oscarsson commented Jan 31, 2025

If you re-base Ill merge :)

General improvements to the docstrings of abstract hardware objects:

* Add an `Emits` custom section for Sphinx's Napoleon extension
* Fix docstrings formatting
* Document some emitted signals
* Move types from docstring to type hints when possible
* Som formatting with `black`

GitHub: supersedes mxcube#866
GitHub: supersedes mxcube#947
@fabcor-maxiv fabcor-maxiv force-pushed the abstract-hwo-docstrings branch from 4a7ccc3 to bb80298 Compare January 31, 2025 13:38
@marcus-oscarsson marcus-oscarsson merged commit 4fb8498 into mxcube:develop Jan 31, 2025
8 checks passed
@fabcor-maxiv fabcor-maxiv deleted the abstract-hwo-docstrings branch January 31, 2025 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants