Add support for NI GPIB-ENET/100 bridge.

pyproject.toml

@@ -23,7 +23,7 @@
     "Programming Language :: Python :: 3.12",
     "Programming Language :: Python :: 3.13",
   ]
-  dependencies = ["pyvisa>=1.15.0", "typing_extensions"]
+  dependencies = ["pyvisa>=1.17.0", "typing_extensions"]
   dynamic = ["version"]
 
   [project.optional-dependencies]

pyvisa_py/gpib.py

@@ -9,68 +9,31 @@
 
 import ctypes  # Used for missing bindings not ideal
 from bisect import bisect
-from typing import Any, Iterator, List, Tuple, Type, Union
+from collections.abc import Iterator
+from typing import Any
 
 from pyvisa import attributes, constants
 from pyvisa.constants import ResourceAttribute, StatusCode
-from pyvisa.rname import GPIBInstr, GPIBIntfc, parse_resource_name
+from pyvisa.rname import GPIBInstr, GPIBIntfc
 
-from . import gpib_constants, prologix
+from . import gpib_constants
 from .common import LOGGER
-from .sessions import Session, UnavailableSession, UnknownAttribute, VISARMSession
+from .sessions import Session, UnavailableSession, UnknownAttribute
 
 
 # NOTE dummy implementation that is overwritten when a GPIB library is found
 # Allow to provide session listing even when no GPIB library is available.
-def _find_listeners() -> Iterator[Tuple[int, int, int]]:
+def _find_listeners() -> Iterator[tuple[int, int, int]]:
     yield from ()
 
 
-@Session.register(constants.InterfaceType.gpib, "INSTR")
-class GPIBSessionDispatch(Session):
-    """Dispatch to the proper class based on prologix._PrologixIntfcSession.boards.
+# Dispatch of ``GPIB::INSTR`` resources (native vs Prologix vs NI-ENET/100
+# bridge) is owned by :mod:`pyvisa_py.gpib_dispatch`, which registers the
+# native driver as a backend resolver. This module therefore no longer
+# registers a ``(gpib, "INSTR")`` class itself.
 
-    Uses the __new__ method to intercept the creation of the instance of a
-    GPIB session.  If parsed.board is found in
-    prologix._PrologixIntfcSession.boards, create an instance of
-    prologix.PrologixInstrSession, otherwise create an instance of GPIBSession.
 
-    """
-
-    def __new__(  # type: ignore[misc]
-        cls,
-        resource_manager_session: VISARMSession,
-        resource_name: str,
-        parsed=None,
-        open_timeout: int | None = None,
-    ) -> Session:
-        newcls: Type
-
-        if parsed is None:
-            parsed = parse_resource_name(resource_name)
-
-        if parsed.board in prologix._PrologixIntfcSession.boards:
-            newcls = prologix.PrologixInstrSession
-        else:
-            newcls = GPIBSession
-
-        return newcls(resource_manager_session, resource_name, parsed, open_timeout)
-
-    @staticmethod
-    def list_resources() -> List[str]:
-        return [
-            "GPIB%d::%d::INSTR" % (board, pad)
-            if sad == gpib_constants.sad.NO_SAD
-            else "GPIB%d::%d::%d::INSTR"
-            % (board, pad, sad - gpib_constants.sad.FIRST_SAD)
-            for board, pad, sad in _find_listeners()
-        ]
-
-    # FIXME when PROLOGIX gain the ability to list resources, we should
-    # include PROLOGIX results.
-
-
-def make_unavailable(msg: str) -> Type:
+def make_unavailable(msg: str) -> type[Any]:
     """Creates a fake session class that raises a ValueError if instantiated.
 
     We can't use Session.register_unavailable() because we need to be able to
@@ -83,7 +46,7 @@ def make_unavailable(msg: str) -> Type:
 
     Returns
     -------
-    Type[Session]
+    type[Session]
         Fake session.
 
     """
@@ -155,7 +118,7 @@ def _inner(self):
 _patch_Gpib()
 
 
-def _find_boards() -> Iterator[Tuple[int, int]]:
+def _find_boards() -> Iterator[tuple[int, int]]:
     """Find GPIB board addresses."""
     for board in range(16):
         try:
@@ -164,7 +127,7 @@ def _find_boards() -> Iterator[Tuple[int, int]]:
             LOGGER.debug("GPIB board %i error in _find_boards(): %s", board, repr(e))
 
 
-def _find_listeners() -> Iterator[Tuple[int, int, int]]:  # type: ignore[no-redef]
+def _find_listeners() -> Iterator[tuple[int, int, int]]:  # type: ignore[no-redef]
     """Find GPIB listeners."""
     for board, boardpad in _find_boards():
         for i in range(31):
@@ -329,7 +292,7 @@ class _GPIBCommon(Session):
 
     # Override parsed to take into account the fact that this class is only used
     # for a specific kind of resource
-    parsed: Union[GPIBIntfc, GPIBInstr]
+    parsed: GPIBIntfc | GPIBInstr
 
     #: Bus wide controller.
     controller: Gpib
@@ -384,7 +347,7 @@ def after_parsing(self) -> None:
 
     def _get_timeout(
         self, attribute: constants.ResourceAttribute
-    ) -> Tuple[int, StatusCode]:
+    ) -> tuple[int, StatusCode]:
         if self.interface:
             gpib_timeout = self.interface.ask(gpib_constants.ask.IbaTMO)
             if gpib_timeout and gpib_timeout < len(TIMETABLE):
@@ -440,7 +403,7 @@ def close(self) -> StatusCode:
         self.controller.close()
         return StatusCode.success
 
-    def read(self, count: int) -> Tuple[bytes, StatusCode]:
+    def read(self, count: int) -> tuple[bytes, StatusCode]:
         """Reads data from device or interface synchronously.
 
         Corresponds to viRead function of the VISA library.
@@ -467,7 +430,7 @@ def read(self, count: int) -> Tuple[bytes, StatusCode]:
 
         return self._read(reader, count, checker, False, None, False, gpib.GpibError)
 
-    def write(self, data: bytes) -> Tuple[int, StatusCode]:
+    def write(self, data: bytes) -> tuple[int, StatusCode]:
         """Writes data to device or interface synchronously.
 
         Corresponds to viWrite function of the VISA library.
@@ -561,7 +524,7 @@ def gpib_control_ren(self, mode: constants.RENLineOperation) -> StatusCode:
 
         return constants.StatusCode.success
 
-    def _get_attribute(self, attribute: ResourceAttribute) -> Tuple[Any, StatusCode]:
+    def _get_attribute(self, attribute: ResourceAttribute) -> tuple[Any, StatusCode]:
         """Get the value for a given VISA attribute for this session.
 
         Use to implement custom logic for attributes.
@@ -738,7 +701,7 @@ class GPIBSession(_GPIBCommon):  # type: ignore[no-redef]
     parsed: GPIBInstr
 
     @staticmethod
-    def list_resources() -> List[str]:
+    def list_resources() -> list[str]:
         return [
             "GPIB%d::%d::INSTR" % (board, pad)
             if sad == gpib_constants.sad.NO_SAD
@@ -791,7 +754,7 @@ def assert_trigger(self, protocol: constants.TriggerProtocol) -> StatusCode:
         except gpib.GpibError as e:
             return convert_gpib_error(e, self.interface.ibsta(), "assert trigger")
 
-    def read_stb(self) -> Tuple[int, StatusCode]:
+    def read_stb(self) -> tuple[int, StatusCode]:
         """Read the device status byte."""
         try:
             return self.interface.serial_poll(), StatusCode.success
@@ -800,7 +763,7 @@ def read_stb(self) -> Tuple[int, StatusCode]:
 
     def _get_attribute(
         self, attribute: constants.ResourceAttribute
-    ) -> Tuple[Any, StatusCode]:
+    ) -> tuple[Any, StatusCode]:
         """Get the value for a given VISA attribute for this session.
 
         Use to implement custom logic for attributes. GPIB::INSTR have the
@@ -896,10 +859,10 @@ class GPIBInterface(_GPIBCommon):
     parsed: GPIBIntfc
 
     @staticmethod
-    def list_resources() -> List[str]:
+    def list_resources() -> list[str]:
         return ["GPIB%d::INTFC" % board for board, pad in _find_boards()]
 
-    def gpib_command(self, command_bytes: bytes) -> Tuple[int, StatusCode]:
+    def gpib_command(self, command_bytes: bytes) -> tuple[int, StatusCode]:
         """Write GPIB command byte on the bus.
 
         Corresponds to viGpibCommand function of the VISA library.
@@ -1002,7 +965,7 @@ def gpib_pass_control(
         status = gpib_lib.ibpct(did)
         return convert_gpib_status(status)
 
-    def _get_attribute(self, attribute: ResourceAttribute) -> Tuple[Any, StatusCode]:
+    def _get_attribute(self, attribute: ResourceAttribute) -> tuple[Any, StatusCode]:
         """Get the value for a given VISA attribute for this session.
 
         Use to implement custom logic for attributes. GPIB::INTFC have the