meerschaum 2.7.5__tar.gz → 2.7.6__tar.gz

{meerschaum-2.7.5/meerschaum.egg-info → meerschaum-2.7.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: meerschaum
-Version: 2.7.5
+Version: 2.7.6
 Summary: Sync Time-Series Pipes with Meerschaum
 Home-page: https://meerschaum.io
 Author: Bennett Meares
@@ -127,7 +127,7 @@ Provides-Extra: sql
 Requires-Dist: numpy>=1.18.5; extra == "sql"
 Requires-Dist: pandas[parquet]>=2.0.1; extra == "sql"
 Requires-Dist: pyarrow>=16.1.0; extra == "sql"
-Requires-Dist: dask[complete]>=2024.5.1; extra == "sql"
+Requires-Dist: dask[complete]>=2024.12.1; extra == "sql"
 Requires-Dist: partd>=1.4.2; extra == "sql"
 Requires-Dist: pytz; extra == "sql"
 Requires-Dist: joblib>=0.17.0; extra == "sql"
@@ -187,7 +187,7 @@ Requires-Dist: valkey>=6.0.0; extra == "api"
 Requires-Dist: numpy>=1.18.5; extra == "api"
 Requires-Dist: pandas[parquet]>=2.0.1; extra == "api"
 Requires-Dist: pyarrow>=16.1.0; extra == "api"
-Requires-Dist: dask[complete]>=2024.5.1; extra == "api"
+Requires-Dist: dask[complete]>=2024.12.1; extra == "api"
 Requires-Dist: partd>=1.4.2; extra == "api"
 Requires-Dist: pytz; extra == "api"
 Requires-Dist: joblib>=0.17.0; extra == "api"
@@ -292,7 +292,7 @@ Requires-Dist: pycparser>=2.21.0; extra == "full"
 Requires-Dist: numpy>=1.18.5; extra == "full"
 Requires-Dist: pandas[parquet]>=2.0.1; extra == "full"
 Requires-Dist: pyarrow>=16.1.0; extra == "full"
-Requires-Dist: dask[complete]>=2024.5.1; extra == "full"
+Requires-Dist: dask[complete]>=2024.12.1; extra == "full"
 Requires-Dist: partd>=1.4.2; extra == "full"
 Requires-Dist: pytz; extra == "full"
 Requires-Dist: joblib>=0.17.0; extra == "full"

{meerschaum-2.7.5 → meerschaum-2.7.6}/meerschaum/_internal/shell/Shell.py

@@ -742,8 +742,7 @@ class Shell(cmd.Cmd):
         """
         from meerschaum import get_connector
         from meerschaum.connectors.parse import parse_instance_keys
-        from meerschaum.utils.warnings import warn, info
-        from meerschaum.utils.misc import remove_ansi
+        from meerschaum.utils.warnings import info
 
         if action is None:
             action = []
@@ -829,7 +828,7 @@ class Shell(cmd.Cmd):
         """
         from meerschaum import get_connector
         from meerschaum.connectors.parse import parse_repo_keys
-        from meerschaum.utils.warnings import warn, info
+        from meerschaum.utils.warnings import info
 
         if action is None:
             action = []
@@ -878,7 +877,6 @@ class Shell(cmd.Cmd):
         
         Note that executors are API instances.
         """
-        from meerschaum import get_connector
         from meerschaum.connectors.parse import parse_executor_keys
         from meerschaum.utils.warnings import warn, info
         from meerschaum.jobs import get_executor_keys_from_context
@@ -894,7 +892,7 @@ class Shell(cmd.Cmd):
             executor_keys = get_executor_keys_from_context()
 
         if executor_keys == 'systemd' and get_executor_keys_from_context() != 'systemd':
-            warn(f"Cannot execute via `systemd`, falling back to `local`...", stack=False)
+            warn("Cannot execute via `systemd`, falling back to `local`...", stack=False)
             executor_keys = 'local'
         
         conn = parse_executor_keys(executor_keys, debug=debug)
@@ -935,7 +933,7 @@ class Shell(cmd.Cmd):
         if args['action'][0] not in shell_attrs['_actions']:
             try:
                 print(textwrap.dedent(getattr(self, f"do_{args['action'][0]}").__doc__))
-            except Exception as e:
+            except Exception:
                 print(f"No help on '{args['action'][0]}'.")
             return ""
         parse_help(args)

{meerschaum-2.7.5 → meerschaum-2.7.6}/meerschaum/_internal/shell/ShellCompleter.py

@@ -7,13 +7,15 @@ Implement the prompt_toolkit Completer base class.
 """
 
 from __future__ import annotations
-from prompt_toolkit.completion import Completer, Completion
-from meerschaum.utils.typing import Optional
-from meerschaum.actions import get_shell, get_completer, get_main_action_name, get_action
+from meerschaum.actions import get_shell, get_main_action_name, get_action
 from meerschaum._internal.arguments import parse_line
 
 from meerschaum.utils.packages import attempt_import, ensure_readline
-prompt_toolkit = attempt_import('prompt_toolkit', lazy=False, install=True)
+
+prompt_toolkit_completion = attempt_import('prompt_toolkit.completion', lazy=False, install=True)
+Completer = prompt_toolkit_completion.Completer
+Completion = prompt_toolkit_completion.Completion
+
 
 class ShellCompleter(Completer):
     """
@@ -30,7 +32,6 @@ class ShellCompleter(Completer):
         yielded = []
         ensure_readline()
         parts = document.text.split('-')
-        ends_with_space = parts[0].endswith(' ')
         last_action_line = parts[0].split('+')[-1]
         part_0_subbed_spaces = last_action_line.replace(' ', '_')
         parsed_text = (part_0_subbed_spaces + '-'.join(parts[1:]))

{meerschaum-2.7.5 → meerschaum-2.7.6}/meerschaum/actions/clear.py

@@ -130,12 +130,15 @@ def _ask_with_rowcounts(
                     )
 
 
-    pipes_rowcounts = {p: p.get_rowcount(begin=begin, end=end, debug=debug) for p in pipes} 
+    pipes_rowcounts = {
+        pipe: pipe.get_rowcount(begin=begin, end=end, debug=debug)
+        for pipe in pipes
+    }
     print_options(
-        [str(p) + f'\n{rc}\n' for p, rc in pipes_rowcounts.items()],
+        [str(pipe) + f'\n{rowcount:,}\n' for pipe, rowcount in pipes_rowcounts.items()],
         header='Number of Rows to be Deleted'
     )
-    total_num_rows = sum([rc for p, rc in pipes_rowcounts.items()])
+    total_num_rows = sum([rowcount for rowcount in pipes_rowcounts.values()])
     question = (
         f"Are you sure you want to delete {total_num_rows:,} rows across {len(pipes)} pipe"
         + ('s' if len(pipes) != 1 else '')

{meerschaum-2.7.5 → meerschaum-2.7.6}/meerschaum/actions/copy.py

@@ -7,7 +7,7 @@ Functions for copying elements.
 """
 
 from __future__ import annotations
-from meerschaum.utils.typing import Union, Any, Sequence, SuccessTuple, Optional, Tuple, List
+from meerschaum.utils.typing import Any, SuccessTuple, Optional, List
 
 def copy(
     action: Optional[List[str]] = None,
@@ -69,44 +69,53 @@ def _copy_pipes(
     Copy pipes' attributes and make new pipes.
     """
     from meerschaum import get_pipes, Pipe
-    from meerschaum.utils.prompt import prompt, yes_no
+    from meerschaum.utils.prompt import prompt, yes_no, get_connectors_completer
     from meerschaum.utils.warnings import warn
     from meerschaum.utils.formatting import print_tuple
     from meerschaum.utils.formatting._shell import clear_screen
     pipes = get_pipes(as_list=True, **kw)
     successes = 0
-    for p in pipes:
-        ck = prompt(f"Connector keys for copy of {p}:", default=p.connector_keys)
-        mk = prompt(f"Metric key for copy of {p}:", default=p.metric_key)
-        lk = prompt(f"Location key for copy of {p} ('None' to omit):", default=str(p.location_key))
+    for pipe in pipes:
+        ck = prompt(
+            f"Connector keys for copy of {pipe}:",
+            default=pipe.connector_keys,
+            completer=get_connectors_completer(),
+        )
+        mk = prompt(f"Metric key for copy of {pipe}:", default=pipe.metric_key)
+        lk = prompt(
+            f"Location key for copy of {pipe} ('None' to omit):",
+            default=str(pipe.location_key),
+        )
         if lk in ('', 'None', '[None]'):
             lk = None
-        _new_pipe = Pipe(
-            ck, mk, lk,
-            parameters=p.parameters.copy(),
-        )
+
         instance_keys = prompt(
-            f"Meerschaum instance to store the new {_new_pipe}:",
-            default=p.instance_keys
+            f"Meerschaum instance for copy of {pipe}:",
+            default=pipe.instance_keys
         )
-        _new_pipe.instance_keys = instance_keys
-        if _new_pipe.get_id(debug=debug) is not None:
-            warn(f"New {_new_pipe} already exists. Skipping...", stack=False)
+        new_pipe = Pipe(
+            ck, mk, lk,
+            instance=instance_keys,
+            parameters=pipe.parameters.copy(),
+        )
+
+        if new_pipe.get_id(debug=debug) is not None:
+            warn(f"{new_pipe} already exists. Skipping...", stack=False)
             continue
-        _register_success_tuple = _new_pipe.register(debug=debug)
+        _register_success_tuple = new_pipe.register(debug=debug)
         if not _register_success_tuple[0]:
-            warn(f"Failed to register new {_new_pipe}.", stack=False)
+            warn(f"Failed to register new {new_pipe}.", stack=False)
             continue
 
         clear_screen(debug=debug)
         successes += 1
         print_tuple(
-            (True, f"Successfully copied attributes of {p} " + f" into {_new_pipe}.")
+            (True, f"Successfully copied attributes of {pipe} " + f" into {new_pipe}.")
         )
         if (
             force or yes_no(
                 (
-                    f"Do you want to copy data from {p} into {_new_pipe}?\n\n"
+                    f"Do you want to copy data from {pipe} into {new_pipe}?\n\n"
                     + "If you specified `--begin`, `--end` or `--params`, data will be filtered."
                 ),
                     noask=noask,
@@ -114,8 +123,8 @@ def _copy_pipes(
                     default='n',
                 )
         ):
-            _new_pipe.sync(
-                p.get_data(
+            new_pipe.sync(
+                pipe.get_data(
                     debug=debug,
                     as_iterator=True,
                     **kw
@@ -142,17 +151,14 @@ def _copy_connectors(
 ) -> SuccessTuple:
     """
     Create a new connector from an existing one.
-
     """
-    import os, pathlib
     from meerschaum.utils.prompt import yes_no, prompt
     from meerschaum.connectors.parse import parse_connector_keys
     from meerschaum.config import _config, get_config
-    from meerschaum.config._edit import write_config
-    from meerschaum.utils.warnings import info, warn
+    from meerschaum.utils.warnings import info
     from meerschaum.utils.formatting import pprint
     from meerschaum.actions import get_action
-    cf = _config()
+    _ = _config()
     if action is None:
         action = []
     if connector_keys is None:
@@ -170,7 +176,7 @@ def _copy_connectors(
 
     try:
         conn = parse_connector_keys(ck)
-    except Exception as e:
+    except Exception:
         return False, f"Unable to parse connector '{ck}'."
 
     if len(_keys) == 2:

{meerschaum-2.7.5 → meerschaum-2.7.6}/meerschaum/actions/sync.py

@@ -275,12 +275,14 @@ def _sync_pipes(
     import time
     import os
     import contextlib
+    from datetime import timedelta
 
     from meerschaum.utils.warnings import warn, info
     from meerschaum.utils.formatting._shell import progress
     from meerschaum.utils.formatting._shell import clear_screen
     from meerschaum.utils.formatting import print_pipes_results
     from meerschaum.config.static import STATIC_CONFIG
+    from meerschaum.utils.misc import interval_str
 
     noninteractive_val = os.environ.get(STATIC_CONFIG['environment']['noninteractive'], None)
     noninteractive = str(noninteractive_val).lower() in ('1', 'true', 'yes')
@@ -321,7 +323,7 @@ def _sync_pipes(
                     for pipe, (_success, _msg) in results_dict.items()
                     if not _success
                 ]
-        except Exception as e:
+        except Exception:
             import traceback
             traceback.print_exc()
             warn(
@@ -351,9 +353,9 @@ def _sync_pipes(
         success_msg = (
             "Successfully spawned threads for pipes:"
             if unblock
-            else f"Successfully synced pipes:"
+            else "Successfully synced pipes:"
         )
-        fail_msg = f"Failed to sync pipes:"
+        fail_msg = "Failed to sync pipes:"
         if results_dict:
             print_pipes_results(
                 results_dict,
@@ -362,8 +364,10 @@ def _sync_pipes(
                 nopretty = nopretty,
             )
 
+        lap_duration_text = interval_str(timedelta(seconds=(lap_end - lap_begin)))
+
         msg = (
-            f"It took {round(lap_end - lap_begin, 2)} seconds to sync " +
+            f"It took {lap_duration_text} to sync " +
             f"{len(success_pipes) + len(failure_pipes)} pipe" +
                 ("s" if (len(success_pipes) + len(failure_pipes)) != 1 else "") + "\n" +
             f"    ({len(success_pipes)} succeeded, {len(failure_pipes)} failed)."
@@ -385,26 +389,26 @@ def _sync_pipes(
 
 
 def _wrap_pipe(
-        pipe,
-        unblock: bool = False,
-        force: bool = False,
-        debug: bool = False,
-        min_seconds: int = 1,
-        workers = None,
-        verify: bool = False,
-        deduplicate: bool = False,
-        bounded: Optional[bool] = None,
-        chunk_interval: Union[timedelta, int, None] = None,
-        **kw
-    ):
+    pipe,
+    unblock: bool = False,
+    force: bool = False,
+    debug: bool = False,
+    min_seconds: int = 1,
+    workers = None,
+    verify: bool = False,
+    deduplicate: bool = False,
+    bounded: Optional[bool] = None,
+    chunk_interval: Union[timedelta, int, None] = None,
+    **kw
+):
     """
     Wrapper function for handling exceptions.
     """
     import time
     import traceback
-    from datetime import datetime, timedelta, timezone
+    from datetime import datetime, timezone
     import meerschaum as mrsm
-    from meerschaum.utils.typing import is_success_tuple, SuccessTuple
+    from meerschaum.utils.typing import is_success_tuple
     from meerschaum.connectors import get_connector_plugin
     from meerschaum.utils.venv import Venv
     from meerschaum.plugins import _pre_sync_hooks, _post_sync_hooks

{meerschaum-2.7.5 → meerschaum-2.7.6}/meerschaum/api/dash/pipes.py

@@ -12,7 +12,6 @@ import shlex
 from textwrap import dedent
 from urllib.parse import urlencode
 
-from dash.dependencies import Input, Output, State
 from meerschaum.utils.typing import List, Optional, Dict, Any, Tuple, Union
 from meerschaum.utils.misc import string_to_dict
 from meerschaum.utils.packages import attempt_import, import_dcc, import_html, import_pandas
@@ -76,7 +75,7 @@ def keys_from_state(
         try:
             #  params = string_to_dict(state['params-textarea.value'])
             params = string_to_dict(state['search-parameters-editor.value'])
-        except Exception as e:
+        except Exception:
             params = None
     else:
         params = None
@@ -506,7 +505,7 @@ def accordion_items_from_pipe(
 
         stats_rows = []
         if rowcount is not None:
-            stats_rows.append(html.Tr([html.Td("Row-count"), html.Td(f"{rowcount}")]))
+            stats_rows.append(html.Tr([html.Td("Row Count"), html.Td(f"{rowcount:,}")]))
         if interval is not None:
             stats_rows.append(
                 html.Tr([html.Td("Timespan"), html.Td(humanfriendly.format_timespan(interval))])

{meerschaum-2.7.5 → meerschaum-2.7.6}/meerschaum/config/_default.py

@@ -76,6 +76,11 @@ default_system_config = {
             },
             'instance': {
                 'stale_temporary_tables_minutes': 1440,
+                'temporary_target': {
+                    'prefix': '_',
+                    'transaction_id_length': 4,
+                    'separator': '_',
+                },
             },
             'chunksize': 100_000,
             'poolclass': 'sqlalchemy.pool.QueuePool',

{meerschaum-2.7.5 → meerschaum-2.7.6}/meerschaum/config/_version.py

@@ -2,4 +2,4 @@
 Specify the Meerschaum release version.
 """
 
-__version__ = "2.7.5"
+__version__ = "2.7.6"

{meerschaum-2.7.5 → meerschaum-2.7.6}/meerschaum/connectors/api/_misc.py

@@ -20,12 +20,13 @@ def get_mrsm_version(self, **kw) -> Optional[str]:
             use_token=False,
             **kw
         ).json()
-    except Exception as e:
+    except Exception:
         return None
     if isinstance(j, dict) and 'detail' in j:
         return None
     return j
 
+
 def get_chaining_status(self, **kw) -> Optional[bool]:
     """
     Fetch the chaining status of the API instance.
@@ -39,7 +40,7 @@ def get_chaining_status(self, **kw) -> Optional[bool]:
         )
         if not response:
             return None
-    except Exception as e:
+    except Exception:
         return None
 
     return response.json()

{meerschaum-2.7.5 → meerschaum-2.7.6}/meerschaum/connectors/api/_pipes.py

@@ -9,8 +9,7 @@ Register or fetch Pipes from the API
 from __future__ import annotations
 import time
 import json
-from io import StringIO
-from datetime import datetime
+from datetime import datetime, timedelta
 
 import meerschaum as mrsm
 from meerschaum.utils.debug import dprint
@@ -178,11 +177,11 @@ def sync_pipe(
     """Sync a DataFrame into a Pipe."""
     from decimal import Decimal
     from meerschaum.utils.debug import dprint
-    from meerschaum.utils.misc import json_serialize_datetime, items_str
+    from meerschaum.utils.misc import json_serialize_datetime, items_str, interval_str
     from meerschaum.config import get_config
     from meerschaum.utils.packages import attempt_import
-    from meerschaum.utils.dataframe import get_numeric_cols, to_json, get_bytes_cols
-    begin = time.time()
+    from meerschaum.utils.dataframe import get_numeric_cols, to_json
+    begin = time.perf_counter()
     more_itertools = attempt_import('more_itertools')
     if df is None:
         msg = f"DataFrame is `None`. Cannot sync {pipe}."
@@ -304,9 +303,10 @@ def sync_pipe(
         num_success_chunks += 1
 
     success_tuple = True, (
-        f"It took {round(time.time() - begin, 2)} seconds to sync {rowcount} row"
+        f"It took {interval_str(timedelta(seconds=(time.perf_counter() - begin)))} "
+        + "to sync {rowcount:,} row"
         + ('s' if rowcount != 1 else '')
-        + f" across {num_success_chunks} chunk" + ('s' if num_success_chunks != 1 else '') +
+        + f" across {num_success_chunks:,} chunk" + ('s' if num_success_chunks != 1 else '') +
         f" to {pipe}."
     )
     return success_tuple
@@ -549,10 +549,9 @@ def create_metadata(
     if debug:
         dprint("Create metadata response: {response.text}")
     try:
-        metadata_response = json.loads(response.text)
+        _ = json.loads(response.text)
     except Exception as e:
         warn(f"Failed to create metadata on {self}:\n{e}")
-        metadata_response = False
     return False
 
 

{meerschaum-2.7.5 → meerschaum-2.7.6}/meerschaum/connectors/sql/_SQLConnector.py

@@ -69,6 +69,7 @@ class SQLConnector(Connector):
         get_pipe_schema,
         create_pipe_table_from_df,
         get_pipe_columns_indices,
+        get_temporary_target,
     )
     from ._plugins import (
         register_plugin,

{meerschaum-2.7.5 → meerschaum-2.7.6}/meerschaum/connectors/sql/_pipes.py

@@ -1505,7 +1505,6 @@ def sync_pipe(
         UPDATE_QUERIES,
         get_reset_autoincrement_queries,
     )
-    from meerschaum.utils.misc import generate_password
     from meerschaum.utils.dtypes import are_dtypes_equal
     from meerschaum.utils.dtypes.sql import get_db_type_from_pd_type
     from meerschaum import Pipe
@@ -1720,9 +1719,7 @@ def sync_pipe(
                 warn(f"Could not reset auto-incrementing primary key for {pipe}.", stack=False)
 
     if update_df is not None and len(update_df) > 0:
-        transact_id = generate_password(3)
-        temp_prefix = '##' if self.flavor != 'oracle' else '_'
-        temp_target = temp_prefix + transact_id + '_' + pipe.target
+        temp_target = self.get_temporary_target(pipe.target, label='update')
         self._log_temporary_tables_creation(temp_target, create=(not pipe.temporary), debug=debug)
         temp_pipe = Pipe(
             pipe.connector_keys.replace(':', '_') + '_', pipe.metric_key, pipe.location_key,
@@ -1743,7 +1740,7 @@ def sync_pipe(
             static=True,
             autoincrement=False,
             parameters={
-                'schema': (self.internal_schema if self.flavor != 'mssql' else None),
+                'schema': self.internal_schema,
                 'hypertable': False,
             },
         )
@@ -1910,15 +1907,18 @@ def sync_pipe_inplace(
     )
     from meerschaum.utils.misc import generate_password
 
-    transact_id = generate_password(3)
-    def get_temp_table_name(label: str) -> str:
-        temp_prefix = '##' if self.flavor != 'oracle' else '_'
-        return temp_prefix + transact_id + '_' + label + '_' + pipe.target
+    transaction_id_length = (
+        mrsm.get_config(
+            'system', 'connectors', 'sql', 'instance', 'temporary_target', 'transaction_id_length'
+        )
+    )
+    transact_id = generate_password(transaction_id_length)
 
     internal_schema = self.internal_schema
+    target = pipe.target
     temp_table_roots = ['backtrack', 'new', 'delta', 'joined', 'unseen', 'update']
     temp_tables = {
-        table_root: get_temp_table_name(table_root)
+        table_root: self.get_temporary_target(target, transact_id=transact_id, label=table_root)
         for table_root in temp_table_roots
     }
     temp_table_names = {
@@ -3626,3 +3626,30 @@ def get_pipe_schema(self, pipe: mrsm.Pipe) -> Union[str, None]:
     A schema string or `None` if nothing is configured.
     """
     return pipe.parameters.get('schema', self.schema)
+
+
+@staticmethod
+def get_temporary_target(
+    target: str,
+    transact_id: Optional[str, None] = None,
+    label: Optional[str] = None,
+    separator: Optional[str] = None,
+) -> str:
+    """
+    Return a unique(ish) temporary target for a pipe.
+    """
+    from meerschaum.utils.misc import generate_password
+    temp_target_cf = (
+        mrsm.get_config('system', 'connectors', 'sql', 'instance', 'temporary_target') or {}
+    )
+    transaction_id_len = temp_target_cf.get('transaction_id_length', 3)
+    transact_id = transact_id or generate_password(transaction_id_len)
+    temp_prefix = temp_target_cf.get('prefix', '_')
+    separator = separator or temp_target_cf.get('separator', '_')
+    return (
+        temp_prefix
+        + target
+        + separator
+        + transact_id
+        + ((separator + label) if label else '')
+    )

{meerschaum-2.7.5 → meerschaum-2.7.6}/meerschaum/connectors/sql/_sql.py

@@ -778,6 +778,7 @@ def to_sql(
     import time
     import json
     from decimal import Decimal
+    from datetime import timedelta
     from meerschaum.utils.warnings import error, warn
     import warnings
     import functools
@@ -816,6 +817,7 @@ def to_sql(
         PD_TO_SQLALCHEMY_DTYPES_FLAVORS,
         get_db_type_from_pd_type,
     )
+    from meerschaum.utils.misc import interval_str
     from meerschaum.connectors.sql._create_engine import flavor_configs
     from meerschaum.utils.packages import attempt_import, import_pandas
     sqlalchemy = attempt_import('sqlalchemy', debug=debug)
@@ -998,7 +1000,13 @@ def to_sql(
 
     end = time.perf_counter()
     if success:
-        msg = f"It took {round(end - start, 2)} seconds to sync {len(df)} rows to {name}."
+        num_rows = len(df)
+        msg = (
+            f"It took {interval_str(timedelta(seconds=(end - start)))} "
+            + f"to sync {num_rows:,} row"
+            + ('s' if num_rows != 1 else '')
+            + f" to {name}."
+        )
     stats['start'] = start
     stats['end'] = end
     stats['duration'] = end - start

{meerschaum-2.7.5 → meerschaum-2.7.6}/meerschaum/jobs/_Job.py

@@ -313,6 +313,7 @@ class Job:
         if not cleanup_success:
             return cleanup_success, cleanup_msg
 
+        _ = self.daemon._properties.pop('result', None)
         return cleanup_success, f"Deleted {self}."
 
     def is_running(self) -> bool:

{meerschaum-2.7.5 → meerschaum-2.7.6}/meerschaum/plugins/__init__.py

@@ -126,8 +126,8 @@ def pre_sync_hook(
 
 
 def post_sync_hook(
-        function: Callable[[Any], Any],
-    ) -> Callable[[Any], Any]:
+    function: Callable[[Any], Any],
+) -> Callable[[Any], Any]:
     """
     Register a function as a sync hook to be executed upon completion of a sync.
     
@@ -143,10 +143,14 @@ def post_sync_hook(
     Examples
     --------
     >>> from meerschaum.plugins import post_sync_hook
+    >>> from meerschaum.utils.misc import interval_str
+    >>> from datetime import timedelta
     >>>
     >>> @post_sync_hook
     ... def log_sync(pipe, success_tuple, duration=None, **kwargs):
-    ...     print(f"It took {round(duration, 2)} seconds to sync {pipe}.")
+    ...     duration_delta = timedelta(seconds=duration)
+    ...     duration_text = interval_str(duration_delta)
+    ...     print(f"It took {duration_text} to sync {pipe}.")
     >>>
     """
     with _locks['_post_sync_hooks']:

{meerschaum-2.7.5 → meerschaum-2.7.6}/meerschaum/utils/daemon/Daemon.py

@@ -1017,7 +1017,8 @@ class Daemon:
 
     def read_pickle(self) -> Daemon:
         """Read a Daemon's pickle file and return the `Daemon`."""
-        import pickle, traceback
+        import pickle
+        import traceback
         if not self.pickle_path.exists():
             error(f"Pickle file does not exist for daemon '{self.daemon_id}'.")
 
@@ -1053,6 +1054,9 @@ class Daemon:
         if self._properties is None:
             self._properties = {}
 
+        if self._properties.get('result', None) is None:
+            _ = self._properties.pop('result', None)
+
         if _file_properties is not None:
             self._properties = apply_patch_to_config(
                 _file_properties,

{meerschaum-2.7.5 → meerschaum-2.7.6}/meerschaum/utils/daemon/__init__.py

@@ -37,7 +37,7 @@ def daemon_entry(sysargs: Optional[List[str]] = None) -> SuccessTuple:
     filtered_sysargs = [arg for arg in sysargs if arg not in ('-d', '--daemon')]
     try:
         label = shlex.join(filtered_sysargs) if sysargs else None
-    except Exception as e:
+    except Exception:
         label = ' '.join(filtered_sysargs) if sysargs else None
 
     name = _args.get('name', None)
@@ -45,7 +45,7 @@ def daemon_entry(sysargs: Optional[List[str]] = None) -> SuccessTuple:
     if name:
         try:
             daemon = Daemon(daemon_id=name)
-        except Exception as e:
+        except Exception:
             daemon = None
 
     if daemon is not None: