8000 (v3.6.12) - Major documentation update by manjavacas · Pull Request #461 · ugr-sail/sinergym · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

(v3.6.12) - Major documentation update #461

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

Merged
merged 34 commits into from
Nov 27, 2024
Merged

(v3.6.12) - Major documentation update #461

merged 34 commits into from
Nov 27, 2024

Conversation

manjavacas
Copy link
Member
@manjavacas manjavacas commented Nov 26, 2024

Description

A complete revision of the Sinergym documentation, as well as of different parts of the repository, has been carried out in order to improve the format and presentation of the tool.

Types of changes

  • Documentation (update in the documentation)

Checklist:

  • I've read the CONTRIBUTION guide (required)
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have reformatted the code using autopep8 second level aggressive.
  • I have reformatted the code using isort.
  • I have ensured cd docs && make spelling && make html pass (required if documentation has been updated.)
  • I have ensured pytest tests/ -vv pass. (required).
  • I have ensured pytype -d import-error sinergym/ pass. (required)

Changelog:

Changelog

  • Comprehensive review of all documentation: improved readability and presentation across most content. Sections have been simplified or expanded, making them easier to read and understand. Spacing and enumerations have also been included throughout the doc sources.
  • Updated page width to enhance the layout of elements such as tables and code snippets.
  • More descriptive and easily referable titles.
  • Sidebar numbering removed for cleaner presentation.
  • Notebooks review and code formatting.
  • Fixed Sphinx compilation warnings.
  • Fixed spelling errors reported by make spelling.

Pending work

  • Add the most-up to date .bib reference.
  • Replace current doc images by the updated images of the paper.
  • Improve the images of the Wrappers docs section.
  • Remove notebooks output warnings.
  • Confirm whether the file tests/test_demo.py is necessary.
  • Clarify the explanations of logging classes.
  • Review docstrings (maybe in a future iteration).

@manjavacas manjavacas added the documentation Improvements or additions to documentation label Nov 26, 2024
@manjavacas
Copy link
Member Author

@AlejandroCN7 Could you lend me a hand with autopep8 and isort check errors? :-)

8000
@AlejandroCN7 AlejandroCN7 changed the title [DOCS] Major documentation update (v3.6.12) - Major documentation update Nov 27, 2024
@AlejandroCN7 AlejandroCN7 merged commit 7b4df5f into ugr-sail:main Nov 27, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0