CLI to install aqua extenstion in Jupyterlab #1203
Conversation
amitkrprajapati
requested review from
darenr,
mayoor,
mrDzurb,
VipulMascarenhas,
qiuosier and
ahosler
as code owners
|
Thank you for your pull request and welcome to our community! To contribute, please sign the Oracle Contributor Agreement (OCA).
To sign the OCA, please create an Oracle account and sign the OCA in Oracle's Contributor Agreement Application. When signing the OCA, please provide your GitHub username. After signing the OCA and getting an OCA approval from Oracle, this PR will be automatically updated. If you are an Oracle employee, please make sure that you are a member of the main Oracle GitHub organization, and your membership in this organization is public. |
oracle-contributor-agreement
bot
added
the
OCA Required
|
📌 Cov diff with main: No lines with coverage information in this diff. 📌 Overall coverage: |
| import subprocess | ||
| from pathlib import Path | ||
|
|
||
| wheel_file_path = Path(path) / f"adsjupyterlab_aqua_extension*.whl" |
There was a problem hiding this comment.
We can introduce the env variable, this would be more flexible. if env variable not provided, then we can use the const path.
| @@ -94,3 +94,12 @@ def _validate_value(flag, value): | |||
| "If you intend to chain a function call to the result, please separate the " | |||
| "flag and the subsequent function call with separator `-`." | |||
| ) | |||
|
|
|||
| @staticmethod | |||
| def install_extension(path = "/ads/extension/"): | |||
There was a problem hiding this comment.
Wouldn't it be better to use ads aqua install, in in future we would need to have many install commands, we can regulate it with the params.
|
📌 Cov diff with main: No lines with coverage information in this diff. 📌 Overall coverage: |
This feature enables easy installation of the Aqua extension from Jupyter Lab.