Skip to content

Commit feb301f

Browse files
author
Gaetano Guerriero
committed
[DEV-5521] Mypy fixes in metrics_controller module
1 parent 45d7460 commit feb301f

File tree

1 file changed

+17
-16
lines changed

1 file changed

+17
-16
lines changed

server/athenian/api/controllers/metrics_controller.py

Lines changed: 17 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -266,8 +266,8 @@ async def compile_filters_prs(
266266
2. The set of all repositories after dereferencing, with service prefixes.
267267
"""
268268
filters = []
269-
checkers = {}
270-
all_repos = set()
269+
checkers: dict[str, AccessChecker] = {}
270+
all_repos: set[str] = set()
271271
for i, for_set in enumerate(for_sets):
272272
repos, prefix, service = await _extract_repos(
273273
request,
@@ -342,8 +342,8 @@ async def _compile_filters_devs(
342342
with service prefixes.
343343
"""
344344
filters = []
345-
checkers = {}
346-
all_repos = set()
345+
checkers: dict[str, AccessChecker] = {}
346+
all_repos: set[str] = set()
347347
for i, for_set in enumerate(for_sets):
348348
repos, prefix, service = await _extract_repos(
349349
request,
@@ -388,8 +388,8 @@ async def compile_filters_checks(
388388
:return: Resulting list of filters.
389389
"""
390390
filters = []
391-
checkers = {}
392-
all_repos = set()
391+
checkers: dict[str, AccessChecker] = {}
392+
all_repos: set[str] = set()
393393
for i, for_set in enumerate(for_sets):
394394
repos, prefix, service = await _extract_repos(
395395
request,
@@ -409,7 +409,7 @@ async def compile_filters_checks(
409409
pusher_groups = (for_set.pusher_groups or []) + (
410410
[for_set.pushers] if for_set.pushers else []
411411
)
412-
teams = set()
412+
teams: set[int] = set()
413413

414414
def ptr(j: int) -> str:
415415
return ".for[%d].%s" % (
@@ -420,7 +420,7 @@ def ptr(j: int) -> str:
420420
for j, pushers in enumerate(pusher_groups):
421421
scan_for_teams(pushers, teams, ptr(j))
422422
teams_map = await fetch_teams_map(teams, account, request.sdb)
423-
commit_author_groups = []
423+
commit_author_groups: list[Sequence[str]] = []
424424
for j, pushers in enumerate(pusher_groups):
425425
if len(
426426
ca_group := compile_developers(
@@ -445,8 +445,8 @@ async def _compile_filters_deployments(
445445
logical_settings: LogicalRepositorySettings,
446446
) -> list[FilterDeployments]:
447447
filters = []
448-
checkers = {}
449-
all_repos = set()
448+
checkers: dict[str, AccessChecker] = {}
449+
all_repos: set[str] = set()
450450
for i, for_set in enumerate(for_sets):
451451
repos, prefix, service = await _extract_repos(
452452
request,
@@ -653,14 +653,15 @@ async def calc_metrics_developers(request: AthenianWebRequest, body: dict) -> we
653653
for granularity, ts, dev_metrics in zip(
654654
filt.granularities, time_intervals, repogroup_metrics,
655655
):
656-
values = []
656+
values: list[list[CalculatedLinearMetricValues]] = []
657657
for ts_metrics in dev_metrics:
658658
values.append(ts_values := [])
659659
for date, metrics in zip(ts, ts_metrics):
660660
metrics = [metrics[i] for i in topic_order]
661-
confidence_mins = [m.confidence_min for m in metrics]
662-
if any(confidence_mins):
663-
confidence_maxs = [m.confidence_max for m in metrics]
661+
metrics_confidence_mins: list = [m.confidence_min for m in metrics]
662+
if any(metrics_confidence_mins):
663+
confidence_mins: list | None = metrics_confidence_mins
664+
confidence_maxs: list | None = [m.confidence_max for m in metrics]
664665
confidence_scores = [m.confidence_score() for m in metrics]
665666
else:
666667
confidence_mins = confidence_maxs = confidence_scores = None
@@ -697,8 +698,8 @@ async def _compile_filters_releases(
697698
list[ReleaseParticipants],
698699
]:
699700
filters = []
700-
checkers = {}
701-
all_repos = set()
701+
checkers: dict[str, AccessChecker] = {}
702+
all_repos: set[str] = set()
702703
prefixer = await Prefixer.load(meta_ids, request.mdb, request.cache)
703704
settings = Settings.from_request(request, account, prefixer)
704705
logical_settings = await settings.list_logical_repositories()

0 commit comments

Comments
 (0)