Skip to content

Conversation

@NoB0
Copy link
Collaborator

@NoB0 NoB0 commented Jul 14, 2025

Update several documentation pages based on the latest progress.

Main changes:

  • Update overview figure
  • Update TOC to include ABUS, LLM-US, and other components
  • Add pages for LLM-US, ABUS and associated NL components

@NoB0 NoB0 requested a review from kbalog July 14, 2025 10:43
@github-actions
Copy link

Current Branch Main Branch
Coverage Badge Coverage Badge

@NoB0 NoB0 mentioned this pull request Jul 17, 2025
Copy link
Contributor

@kbalog kbalog left a comment

Choose a reason for hiding this comment

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

LGTM, with some comments based on submitted paper.

Copy link
Contributor

Choose a reason for hiding this comment

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

Please use the latest version from the paper (minus the color-coding).

Generative NLG
--------------

**Prerequisites**: the LLM used for the generation should be hosted on a ollama server.
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: capitalize Ollama

Btw. isn't it using the common LLM API (incl. OpenAI)?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Currently it uses only Ollama, created an issue to update the class to support all LLM interfaces (issue #213).

@NoB0 NoB0 merged commit a4d266c into main Oct 22, 2025
1 of 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.

3 participants