mandos.model.hit_dfs._to_hits()

Complexity

Conditions

8

Size

Total Lines	29
Code Lines	23

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

from typing import Sequence
def _to_hits(self: AbsDf) -> Sequence[AbstractHit]:
    hits = []
    for row in self.itertuples():
        # noinspection PyUnresolvedReferences
        c = row.hit_class

Variable name "c" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

        # TODO: remove

TODO and FIXME comments should generally be avoided.

        if c == "_DrugbankInteractionHit" and row.data_source == "drugbank:target-functions":
            c = "DrugbankGeneralFunctionHit"

Variable name "c" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

        elif c == "_DrugbankInteractionHit" and row.data_source == "drugbank:targets":
            c = "DrugbankTargetHit"

Variable name "c" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

        try:
            clazz = HIT_CLASSES[c]
        except KeyError:
            raise InjectionError(f"No hit class {c}") from None
        # ignore extra columns
        # if cols are missing, let it fail on clazz.__init__
        data = {f: getattr(row, f) for f in clazz.fields()}
        try:
            # noinspection PyArgumentList
            hit = clazz(**data)
        except ValueError:
            logger.debug(f"Data passed to {clazz}: {data}")
            raise InjectionError(
                f"Fields for {c} do not match:"
                + f" expected {', '.join(clazz.fields())};"
                + f" got {', '.join(data.keys())}"
            )
        hits.append(hit)
    return hits