Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Docker Support and Fix Dependencies for Compatibility #26

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

gustavo-alberto
Copy link

Pull Request: Add Docker Support and Fix Dependencies for Compatibility

While attempting to run the application on Windows, I encountered issues related to missing cmake and specific versions of the required packages. Since my default Python version is the latest, and the last update to the project was in 2020, I implemented a solution to mitigate errors and ensure the project runs as intended according to the versions available at the time of its development.

To avoid environment-related issues (operating system, packages, etc.), I added the ability to run the project via Docker. This ensures consistent execution regardless of the host environment.

Additionally, I fixed the Python version and the dependencies to align with the last stable versions from 2020, adhering to the minor version of semantic versioning, for compatibility.

This closes the issue #25

This commit adds a requirements.txt file with specific package versions to ensure compatibility and correct installation of dependencies. The project hasn't been updated since 2020, so these versions are locked to maintain stability and avoid potential issues with newer releases of the packages.
This commit introduces a `Dockerfile` to containerize the application, making it easier to deploy and run in consistent environments. This will help standardize the development and production setup, ensuring the application works seamlessly across different machines.
This commit updates the path references in Python scripts to ensure compatibility with the new Docker structure. The changes were made to correctly resolve file paths when the application is containerized, preventing errors related to file access in the Docker environment.
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.

1 participant