fix: correct bernoulli_loader kwarg, remove duplicate import, fix mutable default

- Fix bernoulli_loader reading 'dt' instead of 'max_prob' from kwargs
  in encoding/loaders.py. The docstring documents max_prob but the
  code was fetching the wrong key.

- Remove duplicate 'import numpy as np' in network/monitors.py.

- Fix mutable default argument pipeline=[] in
  MulticompartmentConnection.__init__. Changed to None with guard.

Author: haoyu-haoyu

bindsnet/encoding/loaders.py

@@ -26,7 +26,7 @@ def bernoulli_loader(
     :param float max_prob: Maximum probability of spike per Bernoulli trial.
     """
     # Setting kwargs.
-    max_prob = kwargs.get("dt", 1.0)
+    max_prob = kwargs.get("max_prob", 1.0)
 
     for i in range(len(data)):
         # Encode datum as Bernoulli spike trains.

bindsnet/network/monitors.py

@@ -4,10 +4,6 @@
 
 import numpy as np
 import torch
-import numpy as np
-
-from abc import ABC
-from typing import Union, Optional, Iterable, Dict
 
 from bindsnet.network.nodes import Nodes
 from bindsnet.network.topology import (

bindsnet/network/topology.py

@@ -410,7 +410,7 @@ def __init__(
         source: Nodes,
         target: Nodes,
         device: device,
-        pipeline: list = [],
+        pipeline: Optional[list] = None,
         manual_update: bool = False,
         traces: bool = False,
         **kwargs,