Skip to content

Conversation

@asadaaron
Copy link
Collaborator

@asadaaron asadaaron commented Nov 24, 2025

This PR introduces changes from the fix/docker-built-issue-polyphemus-deployment branch.

📝 Summary

📁 Files Changed ( 5 files)

.cursor/.cursorignore
.github/workflows/polyphemus.yml
apps/polyphemus/Dockerfile
apps/polyphemus/src/rhesis/polyphemus/services/services.py
docker-compose.yml

📋 Commit Details

76be1ae8 - update cursorignor and services.py to use userdefined model (Md Asaduzzaman Miah, 2025-11-24 19:09)
9edb63e5 - remove cuda related comment (Md Asaduzzaman Miah, 2025-11-24 18:32)
baa637bf - update the variables and remove the HF_TOKEN and HF_MODEL (Md Asaduzzaman Miah, 2025-11-24 11:37)
d5c7d090 - update docker file and workflow for deployment. (Md Asaduzzaman Miah, 2025-11-24 10:49)

✅ Checklist

  • Code follows the project's style guidelines
  • Self-review of code has been performed
  • Code is commented, particularly in hard-to-understand areas
  • Corresponding changes to documentation have been made
  • Tests have been added/updated for new functionality
  • All tests pass locally

🧪 Testing

📸 Screenshots (if applicable)

🔗 Related Issues


Note

Replaces Polyphemus Dockerfile and CI to build with a predownloaded DEFAULT_MODEL, adds a docker-compose service, and updates generation to report the actual model used.

  • Infrastructure/CI:
    • Update workflow to build with apps/polyphemus/Dockerfile, pass --build-arg DEFAULT_MODEL, and set DEFAULT_MODEL/TRANSFORMERS_CACHE on deploy.
    • Remove HF_MODEL/HF_TOKEN envs from deploy.
  • Docker:
    • Add apps/polyphemus/Dockerfile that installs SDK/app via uv, sets TRANSFORMERS_CACHE, accepts ARG DEFAULT_MODEL, and pre-downloads the model into /app/model_cache.
    • Remove old apps/polyphemus/src/rhesis/polyphemus/Dockerfile.
  • Service logic:
    • In services.py, track requested vs fallback model and return/log the actual model (DEFAULT_MODEL on fallback).
  • Compose:
    • Add polyphemus service with build args/env (DEFAULT_MODEL, TRANSFORMERS_CACHE), healthcheck, and port mapping 8082:8080.

Written by Cursor Bugbot for commit 76be1ae. This will update automatically on new commits. Configure here.

@asadaaron asadaaron self-assigned this Nov 24, 2025
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

This PR is being reviewed by Cursor Bugbot

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@asadaaron asadaaron marked this pull request as draft November 26, 2025 09:26
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.

2 participants