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.

Sign up for GitHub

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

Jump to bottom

New model images #511

Merged
merged 35 commits into from
Mar 2, 2025
Merged

New model images #511

merged 35 commits into from
Mar 2, 2025

Conversation

brifordwylie
Copy link
Member

Pull Request Checklist

Thank you for contributing to the SuperCowPowers Workbench project. Please make sure you've completed the following tasks before submitting your pull request:

PR Tasks

  • Merge 'main' into your PR branch
  • Run tox to run the unit tests
  • Provide brief description of changes

@brifordwylie
adding new model images for training and inference
39282be
@brifordwylie
adding the code/docker for the workbench model image generation (WIP)
8a9a566
@brifordwylie
adding the code/docker for the workbench model image generation (WIP)
d255315
@brifordwylie
adding the code/docker for the workbench model image generation (WIP)
49f0032
@brifordwylie
making a mock_estimator class for testing the training image
8874c43
@brifordwylie
just some cleanup
115d86a
@brifordwylie
cleanup and simplification
72dfacf
@brifordwylie
cleanup and simplification
99a6630
@brifordwylie
removing some old test code
0a06066
@brifordwylie
refactoring inference entry_point and test harness
84d1db8
@brifordwylie
fixing StringIO imports
84378f5
@brifordwylie
improved json handling
5c2d99a
@brifordwylie
change test data a bit
24c34b2
@brifordwylie
changing repo naming
ad4a063
@brifordwylie
changing InferenceImage to ModelImages
091fc3d
@brifordwylie
using new model images
4b837cf
@brifordwylie
unlocking scikit-learn version
2a80c66
@brifordwylie
switching over to 'serve' script
1f6299c
@brifordwylie
cleaning up the requirements.txt files for models since our new train…
1cb2186 
…ing/inference images include these pacakges
@brifordwylie
making the serve script executable
54e5eb3
@brifordwylie
refactoring the training and inference containers
c2f8a37
@brifordwylie
simplifying the inference entry point
5f230e3
@brifordwylie
adding code and metadata to model dir (for pick up by inference conta…
cf008ec 
…iner)
@brifordwylie
changing script args so they don't fail if ENV vars aren't set
fefca78
@brifordwylie
changing script args so they don't fail if ENV vars aren't set
951511b
@brifordwylie
changing logic for copying code files/directories
e56fe74
@brifordwylie
PYTHONPATH doesn't work with importlib, so use sys.path
f424222
@brifordwylie
fixing the file/dir copy from code to model dir
56a59ae
@brifordwylie
flake8/linter cleanup
8b7af33
@brifordwylie
adding install requirements.txt for inference entry point
8868c88
github-advanced-security[bot]
github-advanced-security bot found potential problems Mar 2, 2025
View reviewed changes
@brifordwylie brifordwylie merged commit 6682c1b into main Mar 2, 2025
5 checks passed
@brifordwylie brifordwylie deleted the new_model_images branch March 2, 2025 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@brifordwylie