Skip to content

chore: document application setup (for partners)#101

Merged
mirdono merged 37 commits into
developmentfrom
chore/document-app-setup
May 28, 2026
Merged

chore: document application setup (for partners)#101
mirdono merged 37 commits into
developmentfrom
chore/document-app-setup

Conversation

@mirdono

@mirdono mirdono commented May 18, 2026

Copy link
Copy Markdown
Member

This PR extends the existing documentation to better support partners to set up their own instance of the DECIDe application. This initially focuses on the city of Bamberg who are primarily interested in use cases 0.1 (policy impact report) and 2 (smart search). Further guidelines tailored to other partner cities will be added in follow-up PRs.

Proposed approach

This PR mainly introduces two new files in the new docs folder. The README provides additional information concerning setting up an own, non-development, instance of the DECIDe application.

Furthermore, a Bamberg-specific docker compose override configuration was added. This configuration (partially) disables services that are not relevant for Bamberg and contains some template configuration for relevant services.

Furthermore, the primary README was updated with respect to

  • add links to used services where possible;
  • improved the documentation PDF harvesting pipeline (UC0.0);
  • slightly extended the documentation for UC0.1 to included the used services; and
  • documented UC2 smart search and revised the explanation to set up the smart search in a development stack.

How to test

  • Read all added documentation and assess whether it is factually correct and complete.
  • Add the Bamberg-specific docker compose override to your .env and ensure the stack can be properly started. Note, it is expected that some services do not function correctly due to placeholder values in their configuration.

Notes

In addition to the changes above, this PR also slightly simplifies the docker-compose configuration:

  • Merged smart-search.yml into search.yml as the frontend-smart-search configured in the former relates to the services configured in the latter.
  • Moved the configuration for the report-generation service into base-compose.yml to reduce the number of configuration files.

Related tickets

  • LBRON-1457

@mirdono mirdono added the documentation Improvements or additions to documentation label May 18, 2026
@mirdono mirdono self-assigned this May 18, 2026
@mirdono mirdono force-pushed the chore/document-app-setup branch from da9a33d to 28c9eee Compare May 18, 2026 14:37

@mirdono mirdono left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Added some issues resulting from feedback via rocket.chat. (Along with one that caught my eye while adding these comments.)

Comment thread README.md Outdated
Comment thread docs/README.md
| [Policy impact report](https://github.com/lblod/frontend-decide-policy-impact-report) | `policy-impact-report` |


### Outsource LLM to the cloud

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Issue: Clarify the need to obtain API keys for external services and how they are to be configured.

As part of LBRON-1423 the API keys will become environment variables to be configured in a non-versioned override file. This would be similar as the entity-linking configuration in the added example override.

Furthermore, it is probably best to emphasise these API keys are configured into a local docker-compose.override.yml instead of the versioned example override.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Updated the example override entry for named-entity-recognition based on #43

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Updated the example override entry for pdf-content based on #24

Comment thread docs/README.md
For each service `A` that was added or updated (version bump or changed environment variables), do `docker compose up [-d] A`. For each service `B` for which their configuration was updated in the `../config/B` folder, do a `docker compose restart B`. Note, that `up` on its own does **not** cause a service to update its configuration.


## Service configuration

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Issue: Add a section for the ollama service.

The ollama service should pull in the appropriate LLMs, as described in #55. While the general README mentions this for the smart search setup it is currently not documented this is needed for other use cases as well.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

i flagged this with pedro and joachim, the service should have ollama load the models on startup if they aren't available yet in ollama on disk and crash if that fails. Running manual pulls by drc execing seems crazy

@mirdono

mirdono commented May 19, 2026

Copy link
Copy Markdown
Member Author

Issue: Document municipality-linker service.

In #99 a municipality-linker service was added. Check whether this requires further configuration and/or explanation in the context of this PR.

@mirdono

mirdono commented May 21, 2026

Copy link
Copy Markdown
Member Author

Remark: Add documentation/pointer concerning task/job data model.

The pipelines with their jobs and tasks are central to the application but are not yet explicitly explained here. This came up in the project's get-together today.

Comment thread docs/README.md
Comment thread docs/README.md Outdated
@mirdono

mirdono commented May 22, 2026

Copy link
Copy Markdown
Member Author

Issue: Document municipality-linker service.

In #99 a municipality-linker service was added. Check whether this requires further configuration and/or explanation in the context of this PR.

This service works as is, no partner-specific configuration needed.

@mirdono mirdono force-pushed the chore/document-app-setup branch from 166aa84 to 35f26e5 Compare May 22, 2026 12:33
@mirdono

mirdono commented May 22, 2026

Copy link
Copy Markdown
Member Author

Remark: Add documentation/pointer concerning task/job data model.

The pipelines with their jobs and tasks are central to the application but are not yet explicitly explained here. This came up in the project's get-together today.

Added a short background section in the UC0.0 section. This very shortly introduces the essentials of the pipelines and provides a link to the more elaborate pipeline write up.

@mirdono mirdono force-pushed the chore/document-app-setup branch 2 times, most recently from bd15f83 to 353ad03 Compare May 27, 2026 12:16
@mirdono mirdono force-pushed the chore/document-app-setup branch from 2098bc1 to e292eb7 Compare May 28, 2026 12:14
@mirdono mirdono merged commit 9109184 into development May 28, 2026
@mirdono

mirdono commented May 28, 2026

Copy link
Copy Markdown
Member Author

As discussed on standup of 28 May 2026, merged without explicit review as we want to get it out to the partners.

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