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

doc enhancements #449

Merged
merged 11 commits into from
Dec 12, 2024
Merged

doc enhancements #449

merged 11 commits into from
Dec 12, 2024

Conversation

cbielow
Copy link
Contributor

@cbielow cbielow commented Dec 3, 2024

  • lots of fixes affecting rendering
  • fixes for correctness (referencing the correct lines etc)
  • more elaborate explanation of terms in texts (protein accession etc)
  • sort glossary for easier maintenance
  • update dev build instructions for pyOpenMS
  • glossary fixes: improved descriptions
  • improve QuantData tutorial


DNA, RNA and Protein
********************

We can also work with DNA and RNA sequences in combination with the BioPython
library (you can install BioPython with ``pip install biopython``):

.. code-block:: pseudocode
.. code-block:: python
Copy link
Contributor

Choose a reason for hiding this comment

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

You probably have to be careful. All python code-blocks will be tested. Not sure if this code is functional.
Just saying, you might need to fix it in CI.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I reverted to pseudocode, just to avoid installing biopython, which is rather large

Copy link
Contributor

Choose a reason for hiding this comment

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

yep. fine with me

@cbielow
Copy link
Contributor Author

cbielow commented Dec 4, 2024

building the snippets fails in areas I have not touched.
e.g. https://github.com/OpenMS/pyopenms-docs/actions/runs/12162622521/job/33920001766?pr=449#step:8:84
This probably means that pyopenms-docs has not been build for quite some time.
It seems pyopenms has changed and now breaks the snippets?

@jpfeuffer
Copy link
Contributor

Yes, or they were never touched since CI was introduced.
I think we always just had incremental CI.
No one ever tested everything.

@cbielow
Copy link
Contributor Author

cbielow commented Dec 12, 2024

tests pass.
will merge

@cbielow cbielow merged commit 57c4d18 into OpenMS:master Dec 12, 2024
5 checks passed
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