Skip to content

Commit

Permalink
cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
pedohorse committed Oct 28, 2024
1 parent ad74c35 commit b6fca52
Show file tree
Hide file tree
Showing 27 changed files with 29 additions and 70 deletions.
5 changes: 1 addition & 4 deletions src/lifeblood/basenode.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,16 +4,13 @@
from logging import Logger
from .nodethings import ProcessingResult
from .node_ui import NodeUi
from .node_parameters import ParameterNotFound, Parameter
from .node_parameters import ParameterNotFound, Parameter
from .processingcontext import ProcessingContext
from .logging import get_logger
from .plugin_info import PluginInfo, empty_plugin_info
from .nodegraph_holder_base import NodeGraphHolderBase
from .node_ui_callback_receiver_base import NodeUiCallbackReceiverBase

# reexport
from .nodethings import ProcessingError

from typing import Iterable


Expand Down
5 changes: 0 additions & 5 deletions src/lifeblood/broadcasting.py
Original file line number Diff line number Diff line change
@@ -1,15 +1,10 @@
import asyncio
import socket
from string import ascii_letters
import random
import struct

from . import logging
from .nethelpers import get_localhost
from .defaults import broadcast_port as default_broadcast_port

from . import os_based_cheats

from typing import Tuple, Union, Optional, Callable, Coroutine, Any

Address = Tuple[str, int]
Expand Down
6 changes: 2 additions & 4 deletions src/lifeblood/hardware_resources.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
import psutil
import copy
import re
import json
from .misc import get_unique_machine_id
Expand Down Expand Up @@ -128,11 +126,11 @@ def __repr__(self):
parts = []
for res_name, res in self.__resources.items():
parts.append(f'{res_name}: {res.value}')
for dev_type, dev_res in self.__dev_resources.items():
for dev_type, dev_name, dev_res in self.__dev_resources:
dev_parts = []
for res_name, res in dev_res.items():
dev_parts.append(f'{res_name}: {res.value}')
parts.append(f'device({dev_type})[{", ".join(dev_parts)}]')
parts.append(f'device(type:"{dev_type}" name:"{dev_name}")[{", ".join(dev_parts)}]')

return f'<hwid={self.hwid}, {", ".join(parts)}>'

Expand Down
3 changes: 1 addition & 2 deletions src/lifeblood/local_notifier.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,9 @@
import json
import uuid
from . import broadcasting
from . import logging
from .nethelpers import get_localhost

from typing import Optional, Tuple, Callable, Coroutine, Any
from typing import Optional, Tuple, Callable, Coroutine
from .logging import get_logger


Expand Down
2 changes: 1 addition & 1 deletion src/lifeblood/main_scheduler.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
from .scheduler_config_provider_file import SchedulerConfigProviderFileOverrides
from . import logging

from typing import Iterable, List, Optional, Tuple, Union
from typing import Iterable, List, Tuple, Union


def __construct_plugin_paths(custom_plugins_path: Union[None, str, Path], plugin_search_locations: Iterable[Union[str, Path]]) -> List[Tuple[Path, str]]:
Expand Down
3 changes: 1 addition & 2 deletions src/lifeblood/misc.py
Original file line number Diff line number Diff line change
@@ -1,14 +1,13 @@
import os
import asyncio
import random
import uuid
import time
import psutil
from time import perf_counter
from contextlib import contextmanager, asynccontextmanager
from .logging import get_logger, logging

from typing import List, Optional, Union
from typing import Union


class DummyLock:
Expand Down
2 changes: 0 additions & 2 deletions src/lifeblood/names.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
import re

from lifeblood.logging import get_logger
from typing import Iterable

Expand Down
5 changes: 1 addition & 4 deletions src/lifeblood/net_messages/impl/message_protocol.py
Original file line number Diff line number Diff line change
@@ -1,14 +1,11 @@
import asyncio
import uuid
import struct

from ..logging import get_logger
from ..stream_wrappers import MessageReceiveStream
from ..messages import Message
from ..queue import MessageQueue
from ..address import DirectAddress
from ..exceptions import MessageReceivingError, NoMessageError, MessageTransferError, MessageTransferTimeoutError
from ..interfaces import MessageStreamFactory
from ..exceptions import NoMessageError, MessageTransferError, MessageTransferTimeoutError

from typing import Callable, Awaitable, Tuple

Expand Down
1 change: 0 additions & 1 deletion src/lifeblood/net_messages/impl/tcp_message_processor.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
import asyncio
from ..message_processor import MessageProcessorBase
from ..message_handler import MessageHandlerBase
from ..messages import Message
Expand Down
2 changes: 1 addition & 1 deletion src/lifeblood/net_messages/impl/tcp_message_receiver.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import asyncio
from .message_protocol import MessageProtocol, IProtocolInstanceCounter
from ..interfaces import MessageReceiver, MessageStreamFactory
from ..interfaces import MessageReceiver
from ..messages import Message
from ..address import DirectAddress
from ..logging import get_logger
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
from ..exceptions import MessageTransferError, MessageTransferTimeoutError
from ..interfaces import MessageStreamFactory
from ..stream_wrappers import MessageSendStream, MessageSendStreamBase
from ..address import DirectAddress, AddressChain
from ..address import DirectAddress
from ..defaults import default_stream_timeout
from ..messages import Message

Expand Down
4 changes: 2 additions & 2 deletions src/lifeblood/net_messages/interfaces.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
from .messages import Message
from .message_stream import MessageSendStreamBase, MessageReceiveStreamBase
from .address import DirectAddress, AddressChain
from .message_stream import MessageSendStreamBase
from .address import DirectAddress

from typing import Callable, Awaitable

Expand Down
16 changes: 1 addition & 15 deletions src/lifeblood/net_messages/messages.py
Original file line number Diff line number Diff line change
@@ -1,10 +1,8 @@
import asyncio
import struct
import uuid
from .enums import MessageType
from .address import AddressChain

from typing import Optional, Tuple, Union
from typing import Optional, Union


class MessageInterface:
Expand Down Expand Up @@ -70,17 +68,5 @@ def set_message_destination(self, destination: AddressChain):
def set_message_source(self, source: AddressChain):
self.__source = source

def create_reply_message(self, data: bytes = b''):
if self.__message_type in (MessageType.SESSION_START, MessageType.SESSION_MESSAGE):
return_type = MessageType.SESSION_MESSAGE
elif self.__message_type == MessageType.SESSION_END:
raise RuntimeError('cannot reply to session end message')
elif self.__message_type == MessageType.DEFAULT_MESSAGE:
return_type = self.__message_type
else:
raise RuntimeError(f'unknown message type {self.__message_type}')
return Message(data, return_type, self.__destination, self.__source, self.__session)

def message_type(self) -> MessageType:
return self.__message_type

2 changes: 1 addition & 1 deletion src/lifeblood/net_messages/stream_wrappers.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
from .message_stream import MessageSendStreamBase, MessageReceiveStreamBase
from .enums import MessageType
from .address import AddressChain, DirectAddress
from .exceptions import MessageReceivingError, MessageSendingError, MessageTransferTimeoutError, NoMessageError
from .exceptions import MessageSendingError, MessageTransferTimeoutError, NoMessageError
from .defaults import default_stream_timeout

from typing import Optional, Tuple, Union
Expand Down
3 changes: 0 additions & 3 deletions src/lifeblood/node_visualization_classes.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,4 @@

from typing import Tuple


class NodeColorScheme:
def __init__(self):
self.__main_color = (0, 0, 0)
Expand Down
2 changes: 1 addition & 1 deletion src/lifeblood/pulse_checker.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
from .net_messages.message_processor import MessageProcessorBase
from .net_messages.exceptions import MessageTransferError

from typing import Tuple, Callable, Coroutine
from typing import Callable, Coroutine


class PulseChecker:
Expand Down
2 changes: 1 addition & 1 deletion src/lifeblood/scheduler/data_access.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
from ..worker_resource_definition import WorkerResourceDataType
from ..invocationjob import InvocationResources

from typing import Any, Dict, Iterable, List, Optional, Tuple, Type, Union
from typing import Any, Dict, Iterable, Optional, Tuple, Union

SCHEDULER_DB_FORMAT_VERSION = 5

Expand Down
2 changes: 1 addition & 1 deletion src/lifeblood/scheduler/pinger.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
import time
from .. import logging
from ..worker_message_processor_client import WorkerControlClient
from ..enums import WorkerState, InvocationState, WorkerPingState, WorkerPingReply
from ..enums import WorkerState, WorkerPingState, WorkerPingReply
from .scheduler_component_base import SchedulerComponentBase
from ..net_messages.address import AddressChain
from ..net_messages.exceptions import MessageTransferError, MessageTransferTimeoutError
Expand Down
5 changes: 3 additions & 2 deletions src/lifeblood/scheduler/task_processor.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,12 +17,13 @@
from ..attribute_serialization import serialize_attributes, deserialize_attributes
from ..exceptions import *
from .. import aiosqlite_overlay
from ..ui_events import TaskData, TaskDelta
from ..ui_events import TaskDelta
from ..ui_protocol_data import TaskData
from ..net_messages.address import AddressChain

from .scheduler_component_base import SchedulerComponentBase

from typing import List, Optional, TYPE_CHECKING
from typing import List, TYPE_CHECKING

if TYPE_CHECKING: # TODO: maybe separate a subset of scheduler's methods to smth like SchedulerData class, or idunno, for now no obvious way to separate, so having a reference back
from .scheduler_core import SchedulerCore
Expand Down
4 changes: 2 additions & 2 deletions src/lifeblood/scheduler/ui_state_accessor.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,12 +5,12 @@
import time
from enum import Enum
from ..logging import get_logger
from ..misc import atimeit, aperformance_measurer
from ..misc import aperformance_measurer
from ..enums import InvocationState, TaskState, TaskGroupArchivedState, WorkerState, WorkerType, UIEventType
from ..exceptions import NotSubscribedError
from ..scheduler_event_log import SchedulerEventLog
from ..ui_events import TaskEvent, TaskFullState, TasksUpdated, TasksRemoved, TasksChanged
from ..ui_protocol_data import TaskBatchData, UiData, TaskGroupData, TaskGroupBatchData, TaskGroupStatisticsData, \
from ..ui_protocol_data import TaskBatchData, TaskGroupData, TaskGroupBatchData, TaskGroupStatisticsData, \
NodeGraphStructureData, WorkerBatchData, WorkerData, WorkerResource, WorkerResourceType, WorkerResources, NodeConnectionData, NodeData, TaskData, TaskDelta, \
WorkerDevice, WorkerDeviceResource
from .scheduler_component_base import SchedulerComponentBase
Expand Down
2 changes: 1 addition & 1 deletion src/lifeblood/scheduler_config_provider_file.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
from .config import Config
from .nethelpers import all_interfaces
from .exceptions import SchedulerConfigurationError
from .config import create_default_user_config_file, get_local_scratch_path
from .config import get_local_scratch_path
from .text import escape

from typing import Dict, List, Mapping, Optional, Tuple
Expand Down
2 changes: 1 addition & 1 deletion src/lifeblood/scheduler_event_log.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
from .ui_events import SchedulerEvent
from .enums import UIEventType

from typing import Dict, List, Optional, Set, Tuple
from typing import Dict, List, Optional, Tuple


logger = logging.getLogger(__name__)
Expand Down
7 changes: 1 addition & 6 deletions src/lifeblood/scheduler_task_protocol.py
Original file line number Diff line number Diff line change
@@ -1,16 +1,11 @@
import struct
import asyncio
import aiofiles
from enum import Enum
import pickle
import json

from . import logging
from . import invocationjob
from .taskspawn import TaskSpawn
from .enums import WorkerType, SpawnStatus, WorkerState
from .hardware_resources import HardwareResources
from .worker_metadata import WorkerMetadata
from .enums import SpawnStatus, WorkerState
from .scheduler.scheduler_core import SchedulerCore

from typing import Optional, Tuple
Expand Down
4 changes: 2 additions & 2 deletions src/lifeblood/ui_events.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,11 @@
import struct
from dataclasses import dataclass, field
from .buffered_connection import BufferedReader
from .ui_protocol_data import TaskData, TaskDelta, TaskBatchData, UiData
from .ui_protocol_data import TaskDelta, TaskBatchData
from .buffer_serializable import IBufferSerializable
from .enums import UIEventType

from typing import ClassVar, Dict, Iterable, List, Tuple, Type, Union
from typing import ClassVar, Dict, Iterable, List, Tuple, Type


@dataclass
Expand Down
2 changes: 1 addition & 1 deletion src/lifeblood/ui_protocol_data.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
from dataclasses import dataclass
from enum import Enum

from typing import Dict, List, Tuple, Type, Optional, Set, Union
from typing import Dict, List, Type, Optional, Set, Union


def _serialize_string(s: str, stream: BufferedIOBase) -> int:
Expand Down
2 changes: 1 addition & 1 deletion src/lifeblood/worker_resource_definition.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
from dataclasses import dataclass
from enum import Enum

from typing import Dict, Union, Set, Tuple
from typing import Union, Tuple


class WorkerResourceDataType(Enum):
Expand Down
4 changes: 1 addition & 3 deletions src/lifeblood_testing_common/nodes_common.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,4 @@
import asyncio
import time
from dataclasses import dataclass
import os
import shutil
import tempfile
Expand All @@ -17,7 +15,7 @@
from lifeblood.scheduler.scheduler import Scheduler
from lifeblood_testing_common.common import create_default_scheduler
from lifeblood.worker import Worker
from lifeblood.invocationjob import Invocation, InvocationJob, InvocationResources, Environment
from lifeblood.invocationjob import Invocation, InvocationResources, Environment
from lifeblood.scheduler.pinger import Pinger
from lifeblood.pluginloader import PluginNodeDataProvider
from lifeblood.processingcontext import ProcessingContext
Expand Down

0 comments on commit b6fca52

Please sign in to comment.