Skip to content

Commit

Permalink
Merge pull request #5549 from jenshnielsen/remove_ignore
Browse files Browse the repository at this point in the history
remove a few no longer needed type ignores
  • Loading branch information
jenshnielsen authored Nov 23, 2023
2 parents 4e7565e + 5aae9b1 commit d917e4e
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion src/qcodes/dataset/dond/do_nd.py
Original file line number Diff line number Diff line change
Expand Up @@ -722,7 +722,7 @@ def dond(
for set_events in tqdm(sweeper, disable=not show_progress):
LOG.debug("Processing set events: %s", set_events)
results: dict[ParameterBase, Any] = {}
for set_event in set_events: # pyright: ignore[reportGeneralTypeIssues]
for set_event in set_events:
if set_event.should_set:
set_event.parameter(set_event.new_value)
for act in set_event.actions:
Expand Down
4 changes: 2 additions & 2 deletions src/qcodes/dataset/measurement_extensions.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
from collections.abc import Generator, Sequence
from contextlib import ExitStack, contextmanager
from dataclasses import dataclass
from typing import Any, Callable
from typing import Any

from opentelemetry import trace

Expand Down Expand Up @@ -130,7 +130,7 @@ def parse_dond_into_args(
raise ValueError("dond_into does not support multiple datasets")
elif isinstance(par, ParameterBase) and par.gettable:
params_meas.append(par)
elif isinstance(par, Callable): # type: ignore [arg-type]
elif callable(par):
params_meas.append(par)
return sweep_instances, params_meas

Expand Down

0 comments on commit d917e4e

Please sign in to comment.