Merged
Conversation
…data more realistic
Database operations optimization
…cal procedures generation and fix some bugs
Doctors and departments upgrade
There was a problem hiding this comment.
Pull Request Overview
This PR optimizes Docker builds and refactors the codebase from a doctor-centric model to a department-based personnel system. The changes significantly reduce build time from ~1 minute 40 seconds to ~30 seconds while introducing more complex medical department management.
- Replaced pip with uv package manager for faster dependency installation
- Removed unused audio dependencies and pandas from requirements
- Transitioned from single doctor assignments to department-based personnel management with doctors and nurses
- Updated data models to support multi-department hospital structure with proper personnel assignments
Reviewed Changes
Copilot reviewed 14 out of 14 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| frontend/requirements.txt | Removed pyaudio dependency from elevenlabs package |
| frontend/main.py | Major refactor to support department-based personnel system and updated function signatures |
| frontend/locales/pl/LC_MESSAGES/base.po | Added Polish translations for medical departments and procedures |
| frontend/locales/en/LC_MESSAGES/base.po | Removed unused medical procedure translations |
| frontend/Dockerfile | Replaced pip with uv and removed audio-related system dependencies |
| faker/seed_data.py | Complete rewrite to generate department-based data with personnel assignments |
| faker/models.py | Updated database models to support departments and personnel instead of doctors |
| faker/database_structure_manager.py | Updated to handle new database structure with departments and personnel |
| faker/data_generator.py | Changed from doctor generation to personnel generation with roles |
| faker/Dockerfile | Replaced pip with uv and simplified build process |
| backend/requirements.txt | Removed pandas dependency |
| backend/models.py | Updated models to match new department-based structure |
| backend/main.py | Major refactor to support new data structure and personnel assignments |
| backend/Dockerfile | Replaced pip with uv and simplified build process |
Comments suppressed due to low confidence (1)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Key changes
Reduced building docker images time from ~1min 40s to ~30s