Skip to content

Conversation

@wayangalihpratama
Copy link
Collaborator

@wayangalihpratama wayangalihpratama commented May 5, 2025


@wayangalihpratama wayangalihpratama self-assigned this May 5, 2025
@wayangalihpratama wayangalihpratama linked an issue May 5, 2025 that may be closed by this pull request
@wayangalihpratama wayangalihpratama marked this pull request as draft May 5, 2025 11:24
@wayangalihpratama wayangalihpratama marked this pull request as ready for review May 6, 2025 03:52
Copy link
Member

Choose a reason for hiding this comment

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

Why we have this? if you think it's important or make easier for development (not only you), please add this changes directly to dev.yml

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I created a separate Docker Compose file because some of the ports defined aren't available on my local environment, and also to use a compatible ChromaDB Docker image for my ARM64 architecture.

Copy link
Member

Choose a reason for hiding this comment

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

then move the ChromaDB image to .env

Copy link
Member

Choose a reason for hiding this comment

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

instead of data-pipeline, "script" is more generic

Copy link
Member

Choose a reason for hiding this comment

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

so the path can be script/kb_init_unep.py

@wayangalihpratama wayangalihpratama marked this pull request as draft May 6, 2025 09:21
@wayangalihpratama wayangalihpratama marked this pull request as ready for review May 7, 2025 01:41
Copy link
Member

Choose a reason for hiding this comment

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

then move the ChromaDB image to .env

@dedenbangkit dedenbangkit merged commit 6481295 into main May 7, 2025
1 check 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.

Seed RAG Web UI with UNEP documents

3 participants