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

Contribution: Enhanced Docker Setup with CPU-Compatible Detectron2 and Other Improvements #1

Open
ulan-yisaev opened this issue Aug 29, 2024 · 1 comment
Assignees

Comments

@ulan-yisaev
Copy link

Hi @magicyuan876,

I've been working with the mineru-server and have made several enhancements that I believe could be beneficial to the project:

Key Improvements:

  1. Docker Setup:

    • Added a Dockerfile that sets up Detectron2 for CPU environments, making it easier to run the project as a microservice.
    • Integrated a more efficient way to handle model downloads on application startup, further optimizing the image size.
  2. Logging Enhancements:

    • Improved logging to provide real-time command output for better debugging and monitoring.
  3. Code Refactoring:

    • Applied various code refactoring techniques to ensure the code is more maintainable and production-ready.

I’ve pushed these changes to my fork of the repository here, and I’m happy to discuss any of the changes in more detail.

Would you be interested in reviewing and potentially merging these improvements into the main repository? I’d love to contribute to the project!

Looking forward to your thoughts.

@magicyuan876
Copy link
Owner

@ulan-yisaev 非常感谢,本心只是做个小工具抛砖引玉。期待您的提交,谢谢。

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

When branches are created from issues, their pull requests are automatically linked.

2 participants