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

Documentation updates and bugfixes #171

Merged
merged 28 commits into from
Apr 26, 2024
Merged

Documentation updates and bugfixes #171

merged 28 commits into from
Apr 26, 2024

Conversation

LonnekeScheffer
Copy link
Collaborator

Documentation improvements:

  • Primarily: better documentation for developer tutorials
    --- better intro ("why" should you integrate into immuneML)
    --- rewritten tutorials in more detail
    --- dummy examples of new components (SillyEncoder, Sillyclassifier, etc), with minimal clutter

  • Some re-organisation and revisiting of old docs

  • Improved/updated figures (e.g., explanation of YAML structure, valid YAML component combinations)

  • Reformatting of class docs (bold), added missing docs

  • Changes to e.g., MLMethod and DatasetEncoder interfaces to make integration of new components more streamlined

  • removed unfinished WeightsDistribution report that shouldnt have been there

  • simplified bits of code here and there (e.g. removing unused parameters, cleaning up utility methods)

@LonnekeScheffer
Copy link
Collaborator Author

To do before reopening:

  • resolve KerasSequenceCNN issue on master branch
  • make sure tests for 'silly' methods are not executed on github

@LonnekeScheffer LonnekeScheffer merged commit c492ef7 into master Apr 26, 2024
1 check passed
@LonnekeScheffer LonnekeScheffer deleted the docs_updates branch October 16, 2024 21:37
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.

1 participant