Skip to content

Commit 0a4e681

Browse files
committed
fix(refactor): imports ordering and review fixes
1 parent 1ec0533 commit 0a4e681

File tree

4 files changed

+48
-44
lines changed

4 files changed

+48
-44
lines changed

Utilities/examples/multiple_engines_example.py

Lines changed: 15 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -2,28 +2,28 @@
22
import sys
33
import time
44

5-
from google.protobuf import json_format
6-
from google.protobuf.json_format import MessageToJson
7-
from google.protobuf.json_format import MessageToDict
8-
from fds.protobuf.stach.Package_pb2 import Package
9-
10-
from fds.analyticsapi.engines.api_client import ApiClient
115
from fds.analyticsapi.engines import ComponentSummary
12-
from fds.analyticsapi.engines.configuration import Configuration
6+
from fds.analyticsapi.engines.api.calculations_api import CalculationsApi
137
from fds.analyticsapi.engines.api.components_api import ComponentsApi
148
from fds.analyticsapi.engines.api.configurations_api import ConfigurationsApi
15-
from fds.analyticsapi.engines.api.calculations_api import CalculationsApi
169
from fds.analyticsapi.engines.api.utility_api import UtilityApi
10+
from fds.analyticsapi.engines.api_client import ApiClient
11+
from fds.analyticsapi.engines.configuration import Configuration
1712
from fds.analyticsapi.engines.models.calculation import Calculation
1813
from fds.analyticsapi.engines.models.pa_calculation_parameters import PACalculationParameters
19-
from fds.analyticsapi.engines.models.pa_identifier import PAIdentifier
2014
from fds.analyticsapi.engines.models.pa_date_parameters import PADateParameters
15+
from fds.analyticsapi.engines.models.pa_identifier import PAIdentifier
2116
from fds.analyticsapi.engines.models.spar_calculation_parameters import SPARCalculationParameters
22-
from fds.analyticsapi.engines.models.spar_identifier import SPARIdentifier
2317
from fds.analyticsapi.engines.models.spar_date_parameters import SPARDateParameters
18+
from fds.analyticsapi.engines.models.spar_identifier import SPARIdentifier
2419
from fds.analyticsapi.engines.models.vault_calculation_parameters import VaultCalculationParameters
25-
from fds.analyticsapi.engines.models.vault_identifier import VaultIdentifier
2620
from fds.analyticsapi.engines.models.vault_date_parameters import VaultDateParameters
21+
from fds.analyticsapi.engines.models.vault_identifier import VaultIdentifier
22+
from fds.protobuf.stach.Package_pb2 import Package
23+
24+
from google.protobuf import json_format
25+
from google.protobuf.json_format import MessageToJson
26+
from google.protobuf.json_format import MessageToDict
2727
from urllib3 import Retry
2828

2929
# Copy 'Converting API output to Table Format' snippet to a file with name 'stach_extensions.py' to use below import statement
@@ -63,6 +63,7 @@
6363
# config.proxy = "<proxyUrl>"
6464
config.verify_ssl = False
6565

66+
# Setting configuration to retry api calls on http status codes of 429 and 503.
6667
config.retries = Retry(total=3, status=3, status_forcelist=frozenset([429, 503]), backoff_factor=2, raise_on_status=False)
6768

6869
api_client = ApiClient(config)
@@ -71,7 +72,7 @@
7172

7273
components = components_api.get_pa_components(pa_document_name)
7374
pa_component_desc = ComponentSummary(name=pa_component_name, category=pa_component_category)
74-
component_id = [x for x in list(components.keys()) if components[x] == pa_component_desc][0]
75+
component_id = [id for id in list(components.keys()) if components[id] == pa_component_desc][0]
7576

7677
pa_account_identifier = PAIdentifier(pa_benchmark_sp_50)
7778
pa_accounts = [pa_account_identifier]
@@ -83,7 +84,7 @@
8384

8485
components = components_api.get_spar_components(spar_document_name)
8586
spar_component_desc = ComponentSummary(name=spar_component_name, category=spar_component_category)
86-
component_id = [x for x in list(components.keys()) if components[x] == spar_component_desc][0]
87+
component_id = [id for id in list(components.keys()) if components[id] == spar_component_desc][0]
8788

8889
spar_account_identifier = SPARIdentifier(spar_benchmark_r_1000, spar_benchmark_russell_return_type, spar_benchmark_russell_prefix)
8990
spar_accounts = [spar_account_identifier]
@@ -94,7 +95,7 @@
9495

9596
components = components_api.get_vault_components(vault_document_name)
9697
vault_component_desc = ComponentSummary(name=vault_component_name, category=vault_component_category)
97-
component_id = [x for x in list(components.keys()) if components[x] == vault_component_desc][0]
98+
component_id = [id for id in list(components.keys()) if components[id] == vault_component_desc][0]
9899

99100
vault_account_identifier = VaultIdentifier(vault_default_account)
100101
vault_dates = VaultDateParameters(vault_startdate, vault_enddate, frequency)

Utilities/examples/pa_engine_example.py

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -2,21 +2,21 @@
22
import sys
33
import time
44

5-
from google.protobuf import json_format
6-
from google.protobuf.json_format import MessageToJson
7-
from google.protobuf.json_format import MessageToDict
8-
from fds.protobuf.stach.Package_pb2 import Package
9-
10-
from fds.analyticsapi.engines.api_client import ApiClient
115
from fds.analyticsapi.engines import ComponentSummary
12-
from fds.analyticsapi.engines.configuration import Configuration
13-
from fds.analyticsapi.engines.api.components_api import ComponentsApi
146
from fds.analyticsapi.engines.api.calculations_api import CalculationsApi
7+
from fds.analyticsapi.engines.api.components_api import ComponentsApi
158
from fds.analyticsapi.engines.api.utility_api import UtilityApi
9+
from fds.analyticsapi.engines.api_client import ApiClient
10+
from fds.analyticsapi.engines.configuration import Configuration
1611
from fds.analyticsapi.engines.models.calculation import Calculation
1712
from fds.analyticsapi.engines.models.pa_calculation_parameters import PACalculationParameters
18-
from fds.analyticsapi.engines.models.pa_identifier import PAIdentifier
1913
from fds.analyticsapi.engines.models.pa_date_parameters import PADateParameters
14+
from fds.analyticsapi.engines.models.pa_identifier import PAIdentifier
15+
from fds.protobuf.stach.Package_pb2 import Package
16+
17+
from google.protobuf import json_format
18+
from google.protobuf.json_format import MessageToJson
19+
from google.protobuf.json_format import MessageToDict
2020
from urllib3 import Retry
2121

2222
# Copy 'Converting API output to Table Format' snippet to a file with name 'stach_extensions.py' to use below import statement
@@ -43,6 +43,7 @@
4343
# config.proxy = "<proxyUrl>"
4444
config.verify_ssl = False
4545

46+
# Setting configuration to retry api calls on http status codes of 429 and 503.
4647
config.retries = Retry(total=3, status=3, status_forcelist=frozenset([429, 503]), backoff_factor=2, raise_on_status=False)
4748

4849
api_client = ApiClient(config)
@@ -51,7 +52,7 @@
5152

5253
components = components_api.get_pa_components(pa_document_name)
5354
component_desc = ComponentSummary(name=pa_component_name, category=pa_component_category)
54-
component_id = [x for x in list(components.keys()) if components[x] == component_desc][0]
55+
component_id = [id for id in list(components.keys()) if components[id] == component_desc][0]
5556

5657
pa_account_identifier = PAIdentifier(pa_benchmark_sp_50)
5758
pa_accounts = [pa_account_identifier]

Utilities/examples/spar_engine_example.py

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -2,21 +2,21 @@
22
import sys
33
import time
44

5-
from google.protobuf import json_format
6-
from google.protobuf.json_format import MessageToJson
7-
from google.protobuf.json_format import MessageToDict
8-
from fds.protobuf.stach.Package_pb2 import Package
9-
10-
from fds.analyticsapi.engines.api_client import ApiClient
115
from fds.analyticsapi.engines import ComponentSummary
12-
from fds.analyticsapi.engines.configuration import Configuration
13-
from fds.analyticsapi.engines.api.components_api import ComponentsApi
146
from fds.analyticsapi.engines.api.calculations_api import CalculationsApi
7+
from fds.analyticsapi.engines.api.components_api import ComponentsApi
158
from fds.analyticsapi.engines.api.utility_api import UtilityApi
9+
from fds.analyticsapi.engines.api_client import ApiClient
10+
from fds.analyticsapi.engines.configuration import Configuration
1611
from fds.analyticsapi.engines.models.calculation import Calculation
1712
from fds.analyticsapi.engines.models.spar_calculation_parameters import SPARCalculationParameters
18-
from fds.analyticsapi.engines.models.spar_identifier import SPARIdentifier
1913
from fds.analyticsapi.engines.models.spar_date_parameters import SPARDateParameters
14+
from fds.analyticsapi.engines.models.spar_identifier import SPARIdentifier
15+
from fds.protobuf.stach.Package_pb2 import Package
16+
17+
from google.protobuf import json_format
18+
from google.protobuf.json_format import MessageToJson
19+
from google.protobuf.json_format import MessageToDict
2020
from urllib3 import Retry
2121

2222
# Copy 'Converting API output to Table Format' snippet to a file with name 'stach_extensions.py' to use below import statement
@@ -45,6 +45,7 @@
4545
# config.proxy = "<proxyUrl>"
4646
config.verify_ssl = False
4747

48+
# Setting configuration to retry api calls on http status codes of 429 and 503.
4849
config.retries = Retry(total=3, status=3, status_forcelist=frozenset([429, 503]), backoff_factor=2, raise_on_status=False)
4950

5051
api_client = ApiClient(config)
@@ -53,7 +54,7 @@
5354

5455
components = components_api.get_spar_components(spar_document_name)
5556
component_desc = ComponentSummary(name=spar_component_name, category=spar_component_category)
56-
component_id = [x for x in list(components.keys()) if components[x] == component_desc][0]
57+
component_id = [id for id in list(components.keys()) if components[id] == component_desc][0]
5758

5859
spar_account_identifier = SPARIdentifier(spar_benchmark_r_1000, spar_benchmark_russell_return_type, spar_benchmark_russell_prefix)
5960
spar_accounts = [spar_account_identifier]

Utilities/examples/vault_engine_example.py

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -2,22 +2,22 @@
22
import sys
33
import time
44

5-
from google.protobuf import json_format
6-
from google.protobuf.json_format import MessageToJson
7-
from google.protobuf.json_format import MessageToDict
8-
from fds.protobuf.stach.Package_pb2 import Package
9-
10-
from fds.analyticsapi.engines.api_client import ApiClient
115
from fds.analyticsapi.engines import ComponentSummary
12-
from fds.analyticsapi.engines.configuration import Configuration
6+
from fds.analyticsapi.engines.api.calculations_api import CalculationsApi
137
from fds.analyticsapi.engines.api.components_api import ComponentsApi
148
from fds.analyticsapi.engines.api.configurations_api import ConfigurationsApi
15-
from fds.analyticsapi.engines.api.calculations_api import CalculationsApi
169
from fds.analyticsapi.engines.api.utility_api import UtilityApi
10+
from fds.analyticsapi.engines.api_client import ApiClient
11+
from fds.analyticsapi.engines.configuration import Configuration
1712
from fds.analyticsapi.engines.models.calculation import Calculation
1813
from fds.analyticsapi.engines.models.vault_calculation_parameters import VaultCalculationParameters
19-
from fds.analyticsapi.engines.models.vault_identifier import VaultIdentifier
2014
from fds.analyticsapi.engines.models.vault_date_parameters import VaultDateParameters
15+
from fds.analyticsapi.engines.models.vault_identifier import VaultIdentifier
16+
from fds.protobuf.stach.Package_pb2 import Package
17+
18+
from google.protobuf import json_format
19+
from google.protobuf.json_format import MessageToJson
20+
from google.protobuf.json_format import MessageToDict
2121
from urllib3 import Retry
2222

2323
# Copy 'Converting API output to Table Format' snippet to a file with name 'stach_extensions.py' to use below import statement
@@ -43,6 +43,7 @@
4343
# config.proxy = "<proxyUrl>"
4444
config.verify_ssl = False
4545

46+
# Setting configuration to retry api calls on http status codes of 429 and 503.
4647
config.retries = Retry(total=3, status=3, status_forcelist=frozenset([429, 503]), backoff_factor=2, raise_on_status=False)
4748

4849
api_client = ApiClient(config)
@@ -51,7 +52,7 @@
5152

5253
components = components_api.get_vault_components(vault_document_name)
5354
component_desc = ComponentSummary(name=vault_component_name, category=vault_component_category)
54-
component_id = [x for x in list(components.keys()) if components[x] == component_desc][0]
55+
component_id = [id for id in list(components.keys()) if components[id] == component_desc][0]
5556

5657
vault_account_identifier = VaultIdentifier(vault_default_account)
5758
vault_dates = VaultDateParameters(vault_startdate, vault_enddate, frequency)

0 commit comments

Comments
 (0)