Skip to content

Commit

Permalink
[OPIK-111] - opik configure fixes (api key checking, interactive envi…
Browse files Browse the repository at this point in the history
…ronment handling etc) (#313)

* add unit test

* add unit test part II

* add unit test part III

* add unit test part IV

* add unit test part V

* add unit test part VII

* add unit test chapter VIII

* add unit test chapter IX

* add unit test chapter XI

* add unit test chapter XII

* add mock for env vars and config file

* add is_interactive()

* move configure() to separated module

* move configure() unit tests to separated module

* simplified update config

* add non-interactive mode handling

* add on-premise url handling

* save new config file if force=True or user was prompted for input

* remove outdated unit tests

* fix import

* fix import

* handle cloud/on-premise urls properly

* fix saving/updating session credentials

* make most all methods private

* fix cloud/on-prem url handling

* fix non-interactive mode while using local instance

---------

Co-authored-by: Aliaksandr Kuzmik <[email protected]>
Co-authored-by: Andres Cruz <[email protected]>
  • Loading branch information
3 people authored Sep 30, 2024
1 parent 938ed44 commit 8cf319f
Show file tree
Hide file tree
Showing 9 changed files with 2,039 additions and 1,630 deletions.
2 changes: 1 addition & 1 deletion sdks/python/src/opik/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
from .api_objects.dataset.dataset_item import DatasetItem
from .api_objects.dataset import Dataset
from . import _logging
from .opik_configure import configure
from .configurator.configure import configure
from . import package_version
from .plugins.pytest.decorator import llm_unit
from .evaluation import evaluate
Expand Down
2 changes: 1 addition & 1 deletion sdks/python/src/opik/cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
import click
import questionary

from . import opik_configure
from opik.configurator import configure as opik_configure

__version__: str = "0.0.0+dev"
if __package__:
Expand Down
Empty file.
Loading

0 comments on commit 8cf319f

Please sign in to comment.