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

feat: CaraML SDK - v0 #7

Merged
merged 29 commits into from
Feb 20, 2024
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
b3868ce
feat: add merlin sdk to caraml.models
maniteja6799 Feb 5, 2024
ff1b134
chore: all merlin imports to caraml.models
maniteja6799 Feb 7, 2024
c6ba7ee
chore: all merlin client imports to caraml.generated
maniteja6799 Feb 8, 2024
bea393b
feat: add github workflow for syncing client sdks, rename merlin gene…
maniteja6799 Feb 8, 2024
5dd2306
fix: import errors due to pacage mismatches
maniteja6799 Feb 9, 2024
c795399
chore: modify workflow to clone and generate client using openapi gen…
maniteja6799 Feb 9, 2024
3f18705
Merge pull request #1 from maniteja6799/caraml-sdk
Feb 9, 2024
9305aba
fix: minor config fix in generator
maniteja6799 Feb 9, 2024
db27c2b
fix: minor config fix in generator
maniteja6799 Feb 9, 2024
ef59827
feat: sync generated client sdk
maniteja6799 Feb 9, 2024
509f07b
Merge pull request #2 from maniteja6799/copy-caraml-generated-client-sdk
Feb 9, 2024
a3ff01e
chore: get latest changes from merlin sdk, fix all imports, tests run…
maniteja6799 Feb 9, 2024
3dacb2d
fix: some tests fixed with failing tests down to 48
maniteja6799 Feb 12, 2024
9f6fe77
fix: minor change to remove test generated files
maniteja6799 Feb 12, 2024
8888a81
fix: remove unwanted prints in workflow
maniteja6799 Feb 12, 2024
394fc52
chore: Add Makefile, lint and format all files
maniteja6799 Feb 12, 2024
30b9a13
feat: sync generated client sdk
maniteja6799 Feb 13, 2024
34367a6
Merge pull request #3 from maniteja6799/copy-caraml-generated-client-sdk
Feb 13, 2024
1d4e04f
chore: Add openapi generator version as input to workflow
maniteja6799 Feb 13, 2024
9bc0cfc
chore: minor changes
maniteja6799 Feb 13, 2024
3e5a5a1
chore: minor changes to branch name
maniteja6799 Feb 13, 2024
18d8e43
fix: remove routers client to regenerate
maniteja6799 Feb 13, 2024
e5b76ec
Revert "fix: remove routers client to regenerate"
maniteja6799 Feb 13, 2024
0d32f4e
feat: Add turing wrapper code
maniteja6799 Feb 13, 2024
b9a9f40
feat: Add turing tests, all passing expect one
maniteja6799 Feb 13, 2024
bf34c73
feat: add github action to trigger generate-client-sdk worklfow from …
maniteja6799 Feb 14, 2024
3427a2e
feat: add github action to trigger generate-client-sdk worklfow from …
maniteja6799 Feb 14, 2024
836afe9
feat: add github action to trigger generate-client-sdk worklfow from …
maniteja6799 Feb 14, 2024
95bd15a
feat: add ref as app version in module specific version, can be added…
maniteja6799 Feb 14, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
chore: all merlin client imports to caraml.generated
  • Loading branch information
maniteja6799 committed Feb 8, 2024
commit c6ba7ee774b2aa534baffbd9cc69d708e09ccf6c
2 changes: 1 addition & 1 deletion packages/caraml/models/batch/job.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
# See the License for the specific language governing permissions and
# limitations under the License.

import client
import caraml.generated.merlin_client as client
from enum import Enum

from caraml.models.util import autostr
Expand Down
2 changes: 1 addition & 1 deletion packages/caraml/models/client.py
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@

import urllib3
from caraml_auth.id_token_credentials import get_default_id_token_credentials
from generated.merlin_client import (
from caraml.generated.merlin_client import (
ApiClient,
Configuration,
EndpointApi,
Expand Down
2 changes: 1 addition & 1 deletion packages/caraml/models/endpoint.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
from enum import Enum
from typing import Dict

import client
import caraml.generated.merlin_client as client
from caraml.models.autoscaling import (RAW_DEPLOYMENT_DEFAULT_AUTOSCALING_POLICY,
SERVERLESS_DEFAULT_AUTOSCALING_POLICY,
AutoscalingPolicy, MetricsType)
Expand Down
3 changes: 2 additions & 1 deletion packages/caraml/models/fluent.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,8 @@
from contextlib import contextmanager
from typing import Any, Dict, List, Optional

from client import PredictionJob
from caraml.generated.merlin_client import PredictionJob

from caraml.models.autoscaling import AutoscalingPolicy
from caraml.models.batch.config import PredictionJobConfig
from caraml.models.client import MerlinClient
Expand Down
2 changes: 1 addition & 1 deletion packages/caraml/models/logger.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
# limitations under the License.

from enum import Enum
import client
import caraml.generated.merlin_client as client
from caraml.models.util import autostr
from typing import Optional

Expand Down
2 changes: 1 addition & 1 deletion packages/caraml/models/merlin.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
import click
import warnings
import os
import merlin
import caraml.models as merlin
from caraml.models.model import ModelType
from cookiecutter.main import cookiecutter
from caraml.models.util import valid_name_check
Expand Down
4 changes: 2 additions & 2 deletions packages/caraml/models/model.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,12 +24,12 @@
from time import sleep
from typing import Any, Dict, List, Optional, Tuple, Union

import client
import caraml.generated.merlin_client as client
import docker
import mlflow
import pyprind
import yaml
from client import (
from caraml.generated.merlin_client import (
EndpointApi,
EnvironmentApi,
ModelEndpointsApi,
Expand Down
1 change: 0 additions & 1 deletion packages/caraml/models/transformer.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@

from enum import Enum

import client
import yaml
import json

Expand Down
4 changes: 2 additions & 2 deletions packages/caraml/tests/models/client_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,8 @@

import pytest

import client as cl
from client import ApiClient, Configuration
import caraml.generated.merlin_client as cl
from caraml.generated.merlin_client import ApiClient, Configuration
from caraml.models.client import MerlinClient
from caraml.models.endpoint import Status
from caraml.models.model import Model, ModelType, Project
Expand Down
4 changes: 2 additions & 2 deletions packages/caraml/tests/models/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,8 @@
from requests.packages.urllib3.util.retry import Retry
from urllib3_mock import Responses

import client as cl
from client import ApiClient, Configuration
import caraml.generated.merlin_client as cl
from caraml.generated.merlin_client import ApiClient, Configuration
from caraml.models.model import Model, ModelType, ModelVersion, Project


Expand Down
2 changes: 1 addition & 1 deletion packages/caraml/tests/models/logger_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
# limitations under the License.

import pytest
import client
import caraml.generated.merlin_client as client
from caraml.models.logger import Logger,LoggerMode,LoggerConfig,PredictionLoggerConfig


Expand Down
2 changes: 1 addition & 1 deletion packages/caraml/tests/models/merlin_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
import mlflow
import pytest

import client as cl
import caraml.generated.merlin_client as cl
import caraml.models as merlin
from caraml.models.model import ModelVersion

Expand Down
4 changes: 2 additions & 2 deletions packages/caraml/tests/models/model_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,8 @@
import types
from unittest.mock import patch

import client
import client as cl
import caraml.generated.merlin_client as client
import caraml.generated.merlin_client as cl
import caraml.models as merlin
import pytest
from caraml.models import AutoscalingPolicy, DeploymentMode, MetricsType
Expand Down