Skip to content

Conversation

@Jesshuan
Copy link
Collaborator

Clean, folder archi, doc:

  • 'rag system' README completed with all instructions
  • split two distinct folders for the pipeline scripts
  • restore all 'dev set-up' params for local agnostic dev
  • remove specific database config within the code itself

Fix & improvements :

  • Update Dockerfile for Kotaemon image building
  • Fix Kotaemon bug : file_id filter during retrieval
  • Ollama version fixed to 0.6.3 to prevent deep incompatibility with Kotaemon
  • imporve ingestion versino system

Copy link
Collaborator

@fraboniface fraboniface left a comment

Choose a reason for hiding this comment

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

LGTM, couldn't really test locally because I lack some credentials and GPU support but the changes make sense.

Tests still don't pass but I made some progress:

  • Exclude .ipynb files from pre-commit Ruff analysis. The error message is now much more readable and we can see that all errors come from the policy analysis pipeline.
  • Fixed some errors in kotaemon build (GHA and Dockerfile), but it still fails later in the process. I think it would need its own PR, and anyway we're likely to leave kotaemon altogether, so let's not dwell on this.

This is better than main, so I'll merge.

@fraboniface fraboniface merged commit 93d5b45 into main Nov 18, 2025
0 of 2 checks passed
@fraboniface fraboniface deleted the koteamon-agnostic-base branch November 20, 2025 08:54
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