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

Improved containerization for use on GCP, etc. #1546

Merged
merged 40 commits into from
Oct 16, 2024
Merged

Improved containerization for use on GCP, etc. #1546

merged 40 commits into from
Oct 16, 2024

Conversation

RyanMullins
Copy link
Member

No description provided.

@RyanMullins RyanMullins changed the base branch from main to dev August 12, 2024 15:54
@RyanMullins RyanMullins force-pushed the lit-on-gcp branch 4 times, most recently from 8ae9d55 to cf1b3e6 Compare September 11, 2024 13:16
@RyanMullins RyanMullins force-pushed the lit-on-gcp branch 3 times, most recently from daa6f69 to 821016e Compare September 23, 2024 15:01
RyanMullins and others added 20 commits October 11, 2024 12:39
@RyanMullins
Converting hosted demos images to multi-stage builds
4bf1f81
@RyanMullins
Adding initial dockerfile for GCP use case
1075325
@llcourage @RyanMullins
Update lit-on-gcp requirements.
e80c106
@RyanMullins
Updating date on license declaration
ba1fa38
@RyanMullins
Updating dockerfile for LIT on GCP
5d6cbce
@llcourage @RyanMullins
Add model_server to lit-on-gcp,
1acc868
@RyanMullins
Updating copyright year in Dockerfile
7d8904d
@llcourage @RyanMullins
Add unit test for model_server.
c478f8e
@llcourage @RyanMullins
Add webtest package to lit_on_gcp requirement file.
b9a0b82
@RyanMullins
Adding gunicorn config for model_server.py
55bfc99
@llcourage @RyanMullins
Add unit test for model_server of lit on GCP.
180f68a
@llcourage @RyanMullins
Update pyproject.toml and requirment_test files.
424adce
@llcourage @RyanMullins
Fetch model_name from model_config for lit-on-gcp model server.
ae5825a
@RyanMullins
Fixing KeyError in tests related to default name change
64114d5
@RyanMullins
Fixing requirements and udpating README
1d019c7
@RyanMullins
Adding testing instructions to README
f24b841
@llcourage @RyanMullins
Correct the model name of model_server.
32ec273
@llcourage @RyanMullins
Fix wsgi handler name.
0a4eb3f
@llcourage @RyanMullins
Modify the output to be a list to ensure compatibility with JSON seri…
811dc9d 
…alization.
@RyanMullins
Adopting the generate_model_group_names() API from PD Utils
d442048
@RyanMullins RyanMullins marked this pull request as ready for review October 14, 2024 17:43
@RyanMullins RyanMullins requested a review from llcourage October 14, 2024 17:43
llcourage
llcourage reviewed Oct 14, 2024
View reviewed changes
@bdu91 bdu91 assigned bdu91 and unassigned bdu91 Oct 14, 2024
@bdu91 bdu91 self-requested a review October 14, 2024 19:28
bdu91
bdu91 reviewed Oct 14, 2024
View reviewed changes
Copy link
Collaborator

@bdu91 bdu91 left a comment

Choose a reason for hiding this comment

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

Left a few minor comments.

@PAIR-code PAIR-code deleted a comment from google-cla bot Oct 15, 2024
@copybara-service copybara-service bot merged commit 943270a into dev Oct 16, 2024
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bdu91 bdu91 bdu91 approved these changes

@llcourage llcourage llcourage approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@RyanMullins @bdu91 @llcourage