Skip to content

Commit 12d90d7

Browse files
algattikCopilot
andcommitted
fix(build): satisfy Python and spelling checks
🛠️ - Generated by Copilot Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 1bd53031-98d3-45c5-9c34-a08f1ebfcd6a
1 parent 8b578c9 commit 12d90d7

6 files changed

Lines changed: 8 additions & 17 deletions

File tree

.cspell.json

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,12 +66,14 @@
6666
"perclientserverlabel",
6767
"nvmap",
6868
"pixelformat",
69+
"platlib",
6970
"plottable",
7071
"pollable",
7172
"poutine",
7273
"preds",
7374
"printloglevel",
7475
"prio",
76+
"purelib",
7577
"pyrealsense",
7678
"pyremote",
7779
"pyremotecp",

tests/e2e/_aml.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -218,9 +218,7 @@ def assert_aml_data_asset_exists(
218218
raise AssertionError(f"AzureML data asset {asset_name!r} payload was not a JSON object")
219219
actual_path = payload.get("path")
220220
if actual_path != expected_path:
221-
raise AssertionError(
222-
f"AzureML data asset {asset_name!r} had path {actual_path!r}, expected {expected_path!r}"
223-
)
221+
raise AssertionError(f"AzureML data asset {asset_name!r} had path {actual_path!r}, expected {expected_path!r}")
224222
log_e2e(f"AzureML data asset passed: name={asset_name}, path={actual_path}")
225223

226224

tests/e2e/_mlflow.py

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -272,9 +272,7 @@ def assert_aml_osmo_proxy_has_mlflow_tracking(
272272
}
273273
for name, expected in expected_metrics.items():
274274
if tracking.metrics[name] != expected:
275-
raise AssertionError(
276-
f"MLflow run {run_id!r} had {name}={tracking.metrics[name]!r}, expected {expected!r}"
277-
)
275+
raise AssertionError(f"MLflow run {run_id!r} had {name}={tracking.metrics[name]!r}, expected {expected!r}")
278276
if tracking.metrics["osmo.duration_seconds"] < 0:
279277
raise AssertionError(
280278
f"MLflow run {run_id!r} had negative duration {tracking.metrics['osmo.duration_seconds']!r}"
@@ -306,9 +304,7 @@ def assert_osmo_vla_has_mlflow_tracking(
306304
)
307305
run = runs[0]
308306
if run.data.tags.get("framework") != "groot" or run.data.tags.get("source") != "osmo-train":
309-
raise AssertionError(
310-
f"MLflow run {run.info.run_id!r} had unexpected framework/source tags: {run.data.tags}"
311-
)
307+
raise AssertionError(f"MLflow run {run.info.run_id!r} had unexpected framework/source tags: {run.data.tags}")
312308
required_params = ("BASE_MODEL", "BASE_MODEL_REVISION", "ISAAC_GROOT_REF")
313309
missing_params = [name for name in required_params if not run.data.params.get(name)]
314310
if missing_params:

tests/e2e/test_e2e_aml_osmo_proxy.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -139,9 +139,7 @@ def test_aml_osmo_proxy_e2e(
139139

140140
asset_name = f"osmo-{workflow_id}-output-0"
141141
request.addfinalizer(lambda: archive_aml_data_asset(repo_root, aml_workspace, asset_name))
142-
expected_asset_path = (
143-
f"abfss://{_CONTAINER}@{storage_account}.dfs.core.windows.net/{prefix}/"
144-
)
142+
expected_asset_path = f"abfss://{_CONTAINER}@{storage_account}.dfs.core.windows.net/{prefix}/"
145143
assert_aml_data_asset_exists(
146144
repo_root,
147145
aml_workspace,

training/rl/scripts/runtime_provenance.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,7 @@ def _installed_versions(expected: dict[str, str]) -> tuple[dict[str, str], list[
2828
actual = {}
2929
missing = []
3030
install_roots = {
31-
Path(path).resolve()
32-
for name in ("purelib", "platlib")
33-
if (path := sysconfig.get_path(name)) is not None
31+
Path(path).resolve() for name in ("purelib", "platlib") if (path := sysconfig.get_path(name)) is not None
3432
}
3533
for name in expected:
3634
distributions = list(importlib.metadata.distributions(name=name))

workflows/azureml/osmo-proxy/osmo_proxy.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -711,8 +711,7 @@ def _register_aml_data_assets(
711711

712712
if not all([subscription, resource_group, workspace]):
713713
raise RuntimeError(
714-
"AML_SUBSCRIPTION_ID, AML_RESOURCE_GROUP, and AML_WORKSPACE_NAME are required "
715-
"for data asset registration"
714+
"AML_SUBSCRIPTION_ID, AML_RESOURCE_GROUP, and AML_WORKSPACE_NAME are required for data asset registration"
716715
)
717716

718717
from azure.ai.ml import MLClient

0 commit comments

Comments
 (0)