Skip to content

Commit

Permalink
Fix mypy errors
Browse files Browse the repository at this point in the history
  • Loading branch information
callumforrester committed Aug 1, 2023
1 parent d8dcdd5 commit 5d4020c
Show file tree
Hide file tree
Showing 7 changed files with 10 additions and 38 deletions.
3 changes: 1 addition & 2 deletions src/blueapi/core/context.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@
Callable,
Dict,
Generic,
Iterable,
List,
Optional,
Sequence,
Expand Down Expand Up @@ -63,7 +62,7 @@ class BlueskyContext:

_reference_cache: Dict[Type, Type] = field(default_factory=dict)

def wrap(self, plan: MsgGenerator) -> Iterable[PlanWrapper]:
def wrap(self, plan: MsgGenerator) -> MsgGenerator:
wrapped_plan = functools.reduce(
lambda wrapped, next_wrapper: next_wrapper(wrapped),
self.plan_wrappers,
Expand Down
1 change: 0 additions & 1 deletion src/blueapi/plans/plans.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@
import bluesky.plans as bp
from bluesky.protocols import Movable, Readable
from cycler import Cycler, cycler
from dodal.adsim import AdSimDetector
from scanspec.specs import Spec

from blueapi.core import MsgGenerator
Expand Down
22 changes: 3 additions & 19 deletions src/blueapi/plugins/data_writing.py
Original file line number Diff line number Diff line change
@@ -1,29 +1,13 @@
import itertools
from abc import ABC, abstractmethod, abstractproperty
from collections import deque
from pathlib import Path
from typing import (
Any,
Callable,
Deque,
Dict,
Generic,
Iterable,
List,
Optional,
Protocol,
TypeVar,
runtime_checkable,
)
from typing import Optional

import bluesky.plan_stubs as bps
import requests
from aiohttp import ClientSession
from bluesky.protocols import Movable
from bluesky.utils import Msg, make_decorator
from ophyd.areadetector.filestore_mixins import FileStoreBase
from bluesky.utils import make_decorator

from blueapi.core import BlueskyContext, Device, MsgGenerator, walk_devices
from blueapi.core import MsgGenerator
from blueapi.plugins.data_writing_server import DataCollection

from .data_writing_server import DataCollection, DataCollectionSetupResult
Expand Down
3 changes: 1 addition & 2 deletions src/blueapi/plugins/data_writing_server.py
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
import itertools
import os
from enum import Enum
from pathlib import Path
from typing import Dict, Mapping
from typing import Dict

from fastapi import FastAPI
from pydantic import BaseModel
Expand Down
4 changes: 1 addition & 3 deletions src/blueapi/worker/task.py
Original file line number Diff line number Diff line change
@@ -1,11 +1,9 @@
import functools
import logging
from typing import Any, Iterable, Mapping, Optional
from typing import Any, Mapping, Optional

from pydantic import BaseModel, Field

from blueapi.core import BlueskyContext
from blueapi.core.bluesky_types import MsgGenerator, PlanWrapper
from blueapi.utils import BlueapiBaseModel

LOGGER = logging.getLogger(__name__)
Expand Down
12 changes: 4 additions & 8 deletions tests/plugins/file_writing_detector.py
Original file line number Diff line number Diff line change
@@ -1,12 +1,6 @@
import itertools
import uuid
from pathlib import Path
from typing import List, Optional, Tuple
from typing import List

import h5py as h5
import numpy as np
from ophyd import Component, Device, Signal, SignalRO
from ophyd.sim import EnumSignal, SynGauss, SynSignal, SynSignalRO
from ophyd import Component, Device, Signal

from blueapi.plugins.data_writing import DataCollectionProvider

Expand All @@ -27,5 +21,7 @@ def trigger(self, *args, **kwargs):

def stage(self) -> List[object]:
collection = self._provider.current_data_collection
if collection is None:
raise Exception("No active collection")
self.stage_sigs[self.collection_number] = collection.collection_number
return super().stage()
3 changes: 0 additions & 3 deletions tests/plugins/test_data_writing.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,11 +9,8 @@
run_wrapper,
set_run_key_decorator,
set_run_key_wrapper,
stage_decorator,
stage_wrapper,
)
from bluesky.protocols import Readable
from ophyd.sim import SynAxis

from blueapi.core import DataEvent, MsgGenerator
from blueapi.plugins.data_writing import (
Expand Down

0 comments on commit 5d4020c

Please sign in to comment.