exasol / transformers-extension

An Exasol extension for using state-of-the-art pretrained machine learning models via the Hugging Face Transformers API.
MIT License
2 stars 2 forks source link

Refactoring LanguageContainer Deployer to make it reusable by other extensions #159

Closed tkilias closed 8 months ago

tkilias commented 8 months ago
  1. Inject currently hard-coded values into get_container_file_from_github_release

get_container_file_from_github_release(version: str, project_config: ProjectConfig)



2. Move hard-coded values to https://github.com/exasol/transformers-extension/blob/main/exasol_transformers_extension/deployment/language_container_deployer_cli.py
3. Move https://github.com/exasol/transformers-extension/blob/ec5cabdbd6d510a016ca0f7ebe39221be3386495/exasol_transformers_extension/deployment/language_container_deployer_cli.py#L10C1-L33C1 to new python file, maybe deployment_utils
4. Extract body of language_container_deployer_main into new python file
    - https://github.com/exasol/transformers-extension/blob/ec5cabdbd6d510a016ca0f7ebe39221be3386495/exasol_transformers_extension/deployment/language_container_deployer_cli.py#L80
6. Extract click.options into new python 
    - file https://github.com/exasol/transformers-extension/blob/ec5cabdbd6d510a016ca0f7ebe39221be3386495/exasol_transformers_extension/deployment/language_container_deployer_cli.py#L35
    - Example for defining click options reusable: https://github.com/exasol/integration-test-docker-environment/blob/d3450d947ea88e0d4900beac2a2a0cfd65c3806a/exasol_integration_test_docker_environment/cli/options/build_options.py#L6
    - Example for using reusable click options: https://github.com/exasol/script-languages-container-tool/blob/a45963b14bc228de0bd89d6b9c32dd4433173cd9/exasol_script_languages_container_tool/cli/commands/build.py#L19