Skip to content

Commit 4e8aad8

Browse files
committed
Ruff format
1 parent 3d932d6 commit 4e8aad8

23 files changed

+241
-158
lines changed

python/hopsworks/core/environment_api.py

+15-8
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,13 @@ def __init__(
3232

3333
self._environment_engine = environment_engine.EnvironmentEngine(project_id)
3434

35-
def create_environment(self, name: str, description: Optional[str] = None, base_environment_name: Optional[str] = "python-feature-pipeline", await_creation: Optional[bool] = True) -> environment.Environment:
35+
def create_environment(
36+
self,
37+
name: str,
38+
description: Optional[str] = None,
39+
base_environment_name: Optional[str] = "python-feature-pipeline",
40+
await_creation: Optional[bool] = True,
41+
) -> environment.Environment:
3642
"""Create Python environment for the project
3743
3844
```python
@@ -66,13 +72,14 @@ def create_environment(self, name: str, description: Optional[str] = None, base_
6672
name,
6773
]
6874
headers = {"content-type": "application/json"}
69-
data = {"name": name,
70-
"baseImage": {
71-
"name": base_environment_name,
72-
"description": description
73-
}}
75+
data = {
76+
"name": name,
77+
"baseImage": {"name": base_environment_name, "description": description},
78+
}
7479
env = environment.Environment.from_response_json(
75-
_client._send_request("POST", path_params, headers=headers, data=json.dumps(data)),
80+
_client._send_request(
81+
"POST", path_params, headers=headers, data=json.dumps(data)
82+
),
7683
self._project_id,
7784
self._project_name,
7885
)
@@ -148,4 +155,4 @@ def _delete(self, name):
148155
name,
149156
]
150157
headers = {"content-type": "application/json"}
151-
_client._send_request("DELETE", path_params, headers=headers),
158+
(_client._send_request("DELETE", path_params, headers=headers),)

python/hopsworks/core/secret_api.py

+12-6
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,9 @@ def get_secret(self, name: str, owner: str = None) -> secret.Secret:
7272
"shared",
7373
]
7474

75-
return secret.Secret.from_response_json(_client._send_request("GET", path_params, query_params=query_params))[0]
75+
return secret.Secret.from_response_json(
76+
_client._send_request("GET", path_params, query_params=query_params)
77+
)[0]
7678

7779
def get(self, name: str, owner: str = None) -> str:
7880
"""Get the secret's value.
@@ -90,16 +92,20 @@ def get(self, name: str, owner: str = None) -> str:
9092
return self.get_secret(name=name, owner=owner).value
9193
except RestAPIError as e:
9294
if (
93-
e.response.json().get("errorCode", "") == 160048
94-
and e.response.status_code == 404
95-
and util.is_interactive()
95+
e.response.json().get("errorCode", "") == 160048
96+
and e.response.status_code == 404
97+
and util.is_interactive()
9698
):
97-
secret_input = getpass.getpass(prompt="\nCould not find secret, enter value here to create it: ")
99+
secret_input = getpass.getpass(
100+
prompt="\nCould not find secret, enter value here to create it: "
101+
)
98102
return self.create_secret(name, secret_input).value
99103
else:
100104
raise e
101105

102-
def create_secret(self, name: str, value: str, project: str = None) -> secret.Secret:
106+
def create_secret(
107+
self, name: str, value: str, project: str = None
108+
) -> secret.Secret:
103109
"""Create a new secret.
104110
105111
```python

python/hopsworks/environment.py

+11-9
Original file line numberDiff line numberDiff line change
@@ -133,16 +133,18 @@ def install_wheel(self, path: str, await_installation: Optional[bool] = True):
133133
"packageSource": "WHEEL",
134134
}
135135

136-
library_rest = self._library_api._install(
137-
library_name, self.name, library_spec
138-
)
136+
library_rest = self._library_api._install(library_name, self.name, library_spec)
139137

140138
if await_installation:
141-
return self._environment_engine.await_library_command(self.name, library_name)
139+
return self._environment_engine.await_library_command(
140+
self.name, library_name
141+
)
142142

143143
return library_rest
144144

145-
def install_requirements(self, path: str, await_installation: Optional[bool] = True):
145+
def install_requirements(
146+
self, path: str, await_installation: Optional[bool] = True
147+
):
146148
"""Install libraries specified in a requirements.txt file
147149
148150
```python
@@ -184,12 +186,12 @@ def install_requirements(self, path: str, await_installation: Optional[bool] = T
184186
"packageSource": "REQUIREMENTS_TXT",
185187
}
186188

187-
library_rest = self._library_api._install(
188-
library_name, self.name, library_spec
189-
)
189+
library_rest = self._library_api._install(library_name, self.name, library_spec)
190190

191191
if await_installation:
192-
return self._environment_engine.await_library_command(self.name, library_name)
192+
return self._environment_engine.await_library_command(
193+
self.name, library_name
194+
)
193195

194196
return library_rest
195197

python/hopsworks/job_schedule.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ def __init__(
3030
next_execution_date_time=None,
3131
id=None,
3232
end_date_time=None,
33-
**kwargs
33+
**kwargs,
3434
):
3535
self._id = id
3636
self._start_date_time = (

python/hopsworks/util.py

+3-1
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,8 @@ def get_hostname_replaced_url(sub_path: str):
8181
url_parsed = client.get_instance().replace_public_host(urlparse(href))
8282
return url_parsed.geturl()
8383

84+
8485
def is_interactive():
8586
import __main__ as main
86-
return not hasattr(main, '__file__')
87+
88+
return not hasattr(main, "__file__")

python/hsfs/core/explicit_provenance.py

+1-3
Original file line numberDiff line numberDiff line change
@@ -415,9 +415,7 @@ def default(self, obj):
415415
}
416416
elif isinstance(
417417
obj,
418-
(
419-
storage_connector.StorageConnector
420-
),
418+
(storage_connector.StorageConnector),
421419
):
422420
return {
423421
"name": obj.name,

python/hsfs/core/feature_logging.py

+19-13
Original file line numberDiff line numberDiff line change
@@ -6,25 +6,32 @@
66

77

88
class FeatureLogging:
9-
10-
def __init__(self, id: int,
11-
transformed_features: "feature_group.FeatureGroup",
12-
untransformed_features: "feature_group.FeatureGroup"):
9+
def __init__(
10+
self,
11+
id: int,
12+
transformed_features: "feature_group.FeatureGroup",
13+
untransformed_features: "feature_group.FeatureGroup",
14+
):
1315
self._id = id
1416
self._transformed_features = transformed_features
1517
self._untransformed_features = untransformed_features
1618

1719
@classmethod
18-
def from_response_json(cls, json_dict: Dict[str, Any]) -> 'FeatureLogging':
20+
def from_response_json(cls, json_dict: Dict[str, Any]) -> "FeatureLogging":
1921
from hsfs.feature_group import FeatureGroup # avoid circular import
22+
2023
json_decamelized = humps.decamelize(json_dict)
21-
transformed_features = json_decamelized.get('transformed_log')
22-
untransformed_features = json_decamelized.get('untransformed_log')
24+
transformed_features = json_decamelized.get("transformed_log")
25+
untransformed_features = json_decamelized.get("untransformed_log")
2326
if transformed_features:
2427
transformed_features = FeatureGroup.from_response_json(transformed_features)
2528
if untransformed_features:
26-
untransformed_features = FeatureGroup.from_response_json(untransformed_features)
27-
return cls(json_decamelized.get('id'), transformed_features, untransformed_features)
29+
untransformed_features = FeatureGroup.from_response_json(
30+
untransformed_features
31+
)
32+
return cls(
33+
json_decamelized.get("id"), transformed_features, untransformed_features
34+
)
2835

2936
@property
3037
def transformed_features(self) -> "feature_group.FeatureGroup":
@@ -40,14 +47,13 @@ def id(self) -> str:
4047

4148
def to_dict(self):
4249
return {
43-
'id': self._id,
44-
'transformed_log': self._transformed_features,
45-
'untransformed_log': self._untransformed_features,
50+
"id": self._id,
51+
"transformed_log": self._transformed_features,
52+
"untransformed_log": self._untransformed_features,
4653
}
4754

4855
def json(self) -> Dict[str, Any]:
4956
return json.dumps(self, cls=util.FeatureStoreEncoder)
5057

5158
def __repr__(self):
5259
return self.json()
53-

python/hsfs/core/feature_view_api.py

+10-6
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,6 @@ class FeatureViewApi:
4646
_TRANSFORMED_lOG = "transformed"
4747
_UNTRANSFORMED_LOG = "untransformed"
4848

49-
5049
def __init__(self, feature_store_id: int) -> None:
5150
self._feature_store_id = feature_store_id
5251
self._client = client.get_instance()
@@ -407,7 +406,8 @@ def get_models_provenance(
407406
def enable_feature_logging(
408407
self,
409408
feature_view_name: str,
410-
feature_view_version: int,):
409+
feature_view_version: int,
410+
):
411411
_client = client.get_instance()
412412
path_params = self._base_path + [
413413
feature_view_name,
@@ -420,7 +420,8 @@ def enable_feature_logging(
420420
def pause_feature_logging(
421421
self,
422422
feature_view_name: str,
423-
feature_view_version: int,):
423+
feature_view_version: int,
424+
):
424425
_client = client.get_instance()
425426
path_params = self._base_path + [
426427
feature_view_name,
@@ -434,7 +435,8 @@ def pause_feature_logging(
434435
def resume_feature_logging(
435436
self,
436437
feature_view_name: str,
437-
feature_view_version: int,):
438+
feature_view_version: int,
439+
):
438440
_client = client.get_instance()
439441
path_params = self._base_path + [
440442
feature_view_name,
@@ -448,7 +450,8 @@ def resume_feature_logging(
448450
def materialize_feature_logging(
449451
self,
450452
feature_view_name: str,
451-
feature_view_version: int,):
453+
feature_view_version: int,
454+
):
452455
_client = client.get_instance()
453456
path_params = self._base_path + [
454457
feature_view_name,
@@ -469,7 +472,8 @@ def materialize_feature_logging(
469472
def get_feature_logging(
470473
self,
471474
feature_view_name: str,
472-
feature_view_version: int,):
475+
feature_view_version: int,
476+
):
473477
_client = client.get_instance()
474478
path_params = self._base_path + [
475479
feature_view_name,

0 commit comments

Comments
 (0)