tecio-python 0.3.2__py3-none-any.whl → 0.4.0__py3-none-any.whl

tecio/_dat_write.py

@@ -32,9 +32,12 @@ from ._constants import (
 )
 from ._meta import ZoneMeta
 from ._writer import (
+    PassiveVarsSpec,
     PreparedFEZone,
     PreparedOrderedZone,
     TecplotWriter,
+    VarSharingSpec,
+    convert_face_connections_to_one_based,
     normalize_precision,
     prepare_fe_zone,
     prepare_ordered_zone,
@@ -403,8 +406,8 @@ class TecplotDatWriter(TecplotWriter):
         title: str | None = None,
         variables: list[str] | None = None,
         value_locations: Sequence[ValueLocation] | None = None,
-        passive_vars: Sequence[bool | int] | None = None,
-        var_sharing: Sequence[int] | None = None,
+        passive_vars: PassiveVarsSpec | None = None,
+        var_sharing: VarSharingSpec | None = None,
         solution_time: float = 0.0,
         strand_id: int = 0,
         aux: dict[str, Any] | None = None,
@@ -415,39 +418,62 @@ class TecplotDatWriter(TecplotWriter):
         Zone dimensions are inferred from the shape of the first NODAL array.
 
         Args:
-            data:            One NumPy array per dataset variable. Array shape is used
-                             to infer ``imax``, ``jmax``, and ``kmax``; Fortran
-                             (column-major) order is assumed. Pass ``None`` to write a
-                             zone header only.
-            title:           Zone title. Defaults to ``"IJK_Zone_{current_zone + 1}"``.
-            variables:       Variable name list. Required on the first call when the
-                             file has not been opened yet (lazy-open path); ignored once
-                             the file is already initialised. Default to ``[V1, V2, V3,
-                             ...]`` if not provided in open or zone call.
-            value_locations: Per-variable :class:`~libtecio.ValueLocation`. Defaults to
-                             all ``NODAL``.
-            passive_vars:    Per-variable passive flags. Defaults to all active
-                             (``False``).
-            var_sharing:     Per-variable share-from zone index (1-based). Defaults to
-                             no sharing (all zeros).
-            solution_time:   Solution time for transient data. Use ``0.0`` for static
-                             zones. Default to ``0.0`` if not defined.
-            strand_id:       Strand ID for transient data. Use ``0`` for static
-                             zones. Default to ``0`` (static) if not defined.
-            aux:             Zone-level auxiliary data as ``{name: value}`` string
-                             pairs.
-            datapacking:     Must be :attr:`~tecio.libtecio.DataPacking.BLOCK` (the
-                             default).  :attr:`~tecio.libtecio.DataPacking.POINT` is an
-                             ASCII-only layout and is not supported by the SZL binary
-                             format. Defined only for parity with ASCII writer.
+            data (Sequence[npt.ArrayLike]):
+                One NumPy array per active (non-passive, non-shared) dataset variable,
+                in dataset order. Array shape is used to infer ``imax``, ``jmax``, and
+                ``kmax``; Fortran (column-major) order is assumed. Pass ``None`` to
+                write a zone header only.
+            title (str):
+                Zone title. Defaults to ``"IJK_Zone_{index}"``, this zone's 0-based
+                index.
+            variables (list[str]):
+                Variable name list. Required on the first call when the file has not
+                been opened yet (lazy-open path); ignored once the file is already
+                initialised. Default to ``[V1, V2, V3, ...]`` if not provided in open or
+                zone call.
+            value_locations (Sequence[ValueLocation]):
+                Per-active-variable :class:`~tecio.ValueLocation`.  Defaults to all
+                ``NODAL``.
+            passive_vars (list | tuple | set | None):
+                Variables to mark passive: a list, tuple, or set of variable names, or
+                of 0-based indices, e.g. ``{"x"}`` or ``[3]``. Defaults to all active
+                (``passive_vars=None``).
+            var_sharing (Mapping[int | str, int]):
+                Variables to share from another zone: a ``{variable: source zone}``
+                mapping, name or 0-based index to 0-based zone index, e.g. ``{"x": 0,
+                "y": 0}``. Defaults to no sharing.
+            solution_time (float):
+                Solution time for transient data. Default to ``0.0`` if not defined.
+            strand_id (int):
+                Strand ID for transient data. Use ``0`` for static zones. Default to
+                ``0`` (static) if not defined.
+            aux (dict[str, str | float | int]):
+                Zone-level auxiliary data as ``{name: value}`` string pairs.
+            datapacking (DataPacking | str):
+                In :attr:`~tecio.DataPacking.POINT` format, the values for all variables
+                are given for the first point, then the second point, and so on. In
+                :attr:`~tecio.DataPacking.BLOCK` format, all of the values for the first
+                variable are given in a block, then all of the values for the second
+                variable, then all of the values for the third, and so
+                forth. :attr:`~tecio.DataPacking.BLOCK` format must be used for
+                cell-centered data and polyhedral zones
+                (:attr:`~tecio.ZoneType.FEPOLYGON`/:attr:`~tecio.ZoneType.FEPOLYHEDRAL`),
+                as well as for all binary data. `POINT` and `BLOCK` keyword strings also
+                accepted. Defaults to :attr:`~tecio.DataPacking.BLOCK`.
 
         Raises:
-            NotImplementedError: If *datapacking* is
-                                 :attr:`~tecio.libtecio.DataPacking.POINT`.
-            ValueError:          If I/O operation attempted on closed or None file
-                                 handle.
-            RuntimeError:        If attempting to write variable aux before variables
-                                 are defined.
+            TypeError:
+               If *passive_vars*/*var_sharing* isn't a list/tuple/set/mapping as
+               appropriate, or mixes variable names and indices.
+            ValueError:
+               If I/O operation attempted on closed or None file handle, or a variable
+               is named in both *passive_vars* and *var_sharing*.
+            RuntimeError:
+               If attempting to write variable aux before variables are defined.
+            KeyError:
+               A *passive_vars*/*var_sharing* name doesn't match any dataset variable.
+            IndexError:
+               A *passive_vars*/*var_sharing* index is out of range.
         """
         if isinstance(datapacking, str):
             try:
@@ -458,15 +484,16 @@ class TecplotDatWriter(TecplotWriter):
                     "use DataPacking.BLOCK, DataPacking.POINT, or their "
                     "string equivalents."
                 ) from None
+        new_zone_index = self.current_zone + 1
         if title is None:
-            title = f"IJK_Zone_{self.current_zone + 1}"
+            title = f"IJK_Zone_{new_zone_index}"
         if variables is None:
             variables = [f"V{i}" for i in range(1, len(data) + 1)]
 
         # Open the file if lazily loaded and flush buffered aux data
         if not self._opened:
             self._open(variables)
-        if self.current_zone == 0:
+        if self.current_zone == -1:
             self.flush_aux()
 
         # Convert input data to NumPy arrays
@@ -480,7 +507,7 @@ class TecplotDatWriter(TecplotWriter):
         if value_locations is None:
             value_locations = [ValueLocation.NODAL] * len(arrays)
 
-        prepared: PreparedOrderedZone = prepare_ordered_zone(
+        prepared: PreparedOrderedZone | None = prepare_ordered_zone(
             arrays,
             variable_types,
             value_locations=value_locations,
@@ -490,8 +517,10 @@ class TecplotDatWriter(TecplotWriter):
             meta=self._meta,
             on_error=self._handle_zone_error,
         )
-        passive_vars = prepared.passive_vars
-        var_sharing = prepared.var_sharing
+        if prepared is None:
+            return
+        passive_flags = prepared.passive_vars
+        share_idx = prepared.var_sharing
         imax, jmax, kmax = prepared.imax, prepared.jmax, prepared.kmax
         value_locations_global = prepared.value_locations_global
         variable_types_global = prepared.variable_types_global
@@ -506,8 +535,8 @@ class TecplotDatWriter(TecplotWriter):
             kmax=kmax,
             solution_time=solution_time,
             strand_id=strand_id,
-            passive_vars=passive_vars,
-            var_sharing=var_sharing,
+            passive_vars=passive_flags,
+            var_sharing=share_idx,
             value_locations_global=value_locations_global,
             variable_types_global=variable_types_global,
             aux=aux,
@@ -539,7 +568,7 @@ class TecplotDatWriter(TecplotWriter):
                 _stage_float_array(buf, cast, self._float_fmt)
 
         self._check_handle().write(buf.getvalue())
-        self.current_zone += 1
+        self.current_zone = new_zone_index
 
         # Finally set zone metadata after successfully completing TecIO calls
         self._meta.record_zone(
@@ -552,8 +581,8 @@ class TecplotDatWriter(TecplotWriter):
                 num_aux_items=len(aux) if aux else 0,
                 dimensions=(imax, jmax, kmax),
                 value_locations=tuple(value_locations_global),
-                passive_vars=tuple(bool(p) for p in passive_vars),
-                shared_vars=tuple(int(s) for s in var_sharing),
+                passive_vars=tuple(bool(p) for p in passive_flags),
+                shared_vars=tuple(share_idx),
                 data_types=tuple(variable_types_global),
             )
         )
@@ -564,13 +593,13 @@ class TecplotDatWriter(TecplotWriter):
         self,
         data: Sequence[npt.ArrayLike],
         zone_type: ZoneType,
-        *,
         node_map: npt.ArrayLike | None = None,
+        *,
         title: str | None = None,
         variables: list[str] | None = None,
         value_locations: Sequence[ValueLocation] | None = None,
-        passive_vars: Sequence[bool | int] | None = None,
-        var_sharing: Sequence[int] | None = None,
+        passive_vars: PassiveVarsSpec | None = None,
+        var_sharing: VarSharingSpec | None = None,
         con_sharing: int | None = None,
         face_neighbors: npt.ArrayLike | None = None,
         face_neighbor_mode: FaceNeighborMode | None = None,
@@ -589,85 +618,104 @@ class TecplotDatWriter(TecplotWriter):
             ``FEPOLYGON`` and ``FEPOLYHEDRON`` raise :exc:`NotImplementedError`.
 
         Args:
-            data:            Sequence of 1-D arrays, one per dataset variable. NODAL
-                             arrays must have length ``num_nodes``; CELL_CENTERED arrays
-                             must have length ``num_cells``.  ``num_nodes`` and
-                             ``num_cells`` are inferred from ``node_map`` (or from the
-                             ``con_sharing`` source zone when ``node_map`` is omitted).
-            zone_type:       FE zone type from the ZoneType enum. Must be one of the
-                             types in ``_FE_SIMPLE``.
-            node_map:        Integer array of shape ``(num_cells, nodes_per_cell)``
-                             containing 1-based node indices.  32- or 64-bit write is
-                             chosen automatically based on the maximum index value.
-                             Required unless ``con_sharing`` is set, in which case the
-                             connectivity -- and the node/cell counts derived from it --
-                             are inherited from the source zone instead.
-            title:           Zone title string. Defaults to ``"FE_Zone_{current_zone +
-                             1}"`` if not provided.
-            variables:       Variable name list. Required only when the file has not
-                             been opened yet (lazy-open path). Ignored on subsequent
-                             zones once the file is already initialised. Default to
-                             ``[V1, V2, V3, ...]`` if not provided in open or zone call.
-            value_locations: Per-variable ValueLocation. Defaults to all NODAL.
-            passive_vars:    Per-variable passive flags. Defaults to all active
-                             (False).
-            var_sharing:     Per-variable share from zone index. Defaults to no
-                             sharing. Cross-checked against ``node_map`` /
-                             ``con_sharing`` for a consistent node/cell count.
-            con_sharing:     Optional zone index that the connectivity is shared from.
-                             ``None`` or ``0`` indicates no sharing (this zone owns its
-                             connectivity). The first zone in a dataset must own its
-                             connectivity. Connectivity cannot be shared when face
-                             neighbor mode is set to global. Connectivity cannot be
-                             shared between cell-based and face-based finite element
-                             zones.
-            face_neighbors:  Optional face-neighbor connectivity, always a
-                             flat array (whatever shape you pass is
-                             flattened before writing). The number of
-                             values per connection depends on
-                             ``face_neighbor_mode``, a fixed 3 or 4 for the
-                             one-to-one modes, a variable, self-describing
-                             count per record for the "many" modes. The
-                             connection count is always derived from this
-                             array, never a separate parameter.
-            face_neighbor_mode: None (the default) means no face-neighbor
-                             data, matching the read side. If
-                             ``face_neighbors`` is given without this,
-                             treated as LOCAL_ONE_TO_ONE. Given *without*
-                             ``face_neighbors``, raises, that combination is
-                             almost certainly a mistake.
-            face_neighbors_complete: Whether ``face_neighbors`` is the
-                             entire adjacency picture (``True``), Tecplot
-                             shouldn't auto-detect anything further, or a
-                             supplement to auto-detected conformal adjacency
-                             (``False``). ``None`` (the default) omits the
-                             ``FEFACENEIGHBORSCOMPLETE`` keyword entirely,
-                             deferring to Tecplot's own default. ASCII-only:
-                             neither the classic nor new C API can express
-                             this, so it has no equivalent on
-                             :class:`~tecio.TecplotSzlWriter` or
-                             :class:`~tecio.TecplotPltWriter`.
-            solution_time:   Solution time for transient data (0.0 = static).
-            strand_id:       Strand ID for transient data (0 = static).
-            aux:             Zone-level auxiliary data as ``{name: value}`` strings.
-            datapacking:     Must be :attr:`~tecio.libtecio.DataPacking.BLOCK` (the
-                             default).  :attr:`~tecio.libtecio.DataPacking.POINT` is an
-                             ASCII-only layout and is not supported by the SZL binary
-                             format. Defined only for parity with ASCII writer.
+            data (Sequence[npt.ArrayLike]):
+                Sequence of 1-D arrays, one per active (non-passive, non-shared) dataset
+                variable, in dataset order. NODAL arrays must have length ``num_nodes``;
+                CELL_CENTERED arrays must have length ``num_cells``.  ``num_nodes`` and
+                ``num_cells`` are inferred from ``node_map`` (or from the
+                ``con_sharing`` source zone when ``node_map`` is omitted).
+            zone_type (ZoneType):
+                FE zone type from the ZoneType enum. Must be one of the types in
+                ``_FE_SIMPLE``.
+            node_map (npt.ArrayLike):
+                Integer array of shape ``(num_cells, nodes_per_cell)`` containing
+                0-based node indices. Required unless ``con_sharing`` is set, in which
+                case the connectivity are inherited from the source zone instead.
+            title (str):
+                Zone title string. Defaults to ``"FE_Zone_{index}"``, this zone's
+                0-based index.
+            variables (list[str]):
+                Variable name list. Required only when the file has not been opened yet
+                (lazy-open path). Ignored on subsequent zones once the file is already
+                initialised. Default to ``[V1, V2, V3, ...]`` if not provided in open or
+                zone call.
+            value_locations (Sequence[ValueLocation]):
+                Per-active-variable :class:`~tecio.ValueLocation`. Defaults to all
+                ``NODAL``.
+            passive_vars (list | tuple | set | None):
+                Variables to mark passive: a list, tuple, or set of variable names, or
+                of 0-based indices, e.g. ``{"x"}`` or ``[3]``. Defaults to all active
+                (``passive_vars=None``).
+            var_sharing (Mapping[int | str, int]):
+                Variables to share from another zone: a ``{variable: source zone}``
+                mapping, name or 0-based index to 0-based zone index. Defaults to no
+                sharing (None or an empty mapping). Cross-checked against ``node_map`` /
+                ``con_sharing`` for a consistent node/cell count.
+            con_sharing (int):
+                Optional zone index that the connectivity is shared from. ``None``
+                indicates no sharing (this zone owns its connectivity). The first
+                zone in a dataset must own its connectivity. Connectivity cannot be
+                shared when face neighbor mode is set to global. Connectivity cannot be
+                shared between cell-based and face-based finite element zones.
+            face_neighbors (npt.ArrayLike):
+                Optional face-neighbor connectivity, always a flat array (whatever shape
+                you pass is flattened before writing). The number of values per
+                connection depends on ``face_neighbor_mode``, a fixed 3 or 4 for the
+                one-to-one modes, a variable, self-describing count per record for the
+                "many" modes. The connection count is always derived from this array,
+                never a separate parameter.
+            face_neighbor_mode (FaceNeighborMode):
+                Default to None meaning no face-neighbor data. If ``face_neighbors`` is
+                supplied without defining a face neighbor mode, it is treated as
+                LOCAL_ONE_TO_ONE. Cannot be defined without also supplying a
+                ``face_neighbors`` array.
+            face_neighbors_complete (bool):
+                Whether ``face_neighbors`` is the entire adjacency picture (``True``),
+                Tecplot shouldn't auto-detect anything further, or a supplement to
+                auto-detected conformal adjacency (``False``). ``None`` (the default)
+                omits the ``FEFACENEIGHBORSCOMPLETE`` keyword entirely, deferring to the
+                Tecplot default. ASCII-only: neither the classic nor new C API can
+                express this, so it has no equivalent on
+                :class:`~tecio.TecplotSzlWriter` or :class:`~tecio.TecplotPltWriter`.
+            solution_time (float):
+                Solution time for transient data. Default to ``0.0`` if not defined.
+            strand_id (int):
+                Strand ID for transient data. Use ``0`` for static zones. Default to
+                ``0`` (static) if not defined.
+            aux (dict[str, str | float | int]):
+                Zone-level auxiliary data as ``{name: value}`` string pairs.
+            datapacking (DataPacking | str):
+                In :attr:`~tecio.DataPacking.POINT` format, the values for all variables
+                are given for the first point, then the second point, and so on. In
+                :attr:`~tecio.DataPacking.BLOCK` format, all of the values for the first
+                variable are given in a block, then all of the values for the second
+                variable, then all of the values for the third, and so
+                forth. :attr:`~tecio.DataPacking.BLOCK` format must be used for
+                cell-centered data and polyhedral zones
+                (:attr:`~tecio.ZoneType.FEPOLYGON`/:attr:`~tecio.ZoneType.FEPOLYHEDRAL`),
+                as well as for all binary data. `POINT` and `BLOCK` keyword strings also
+                accepted. Defaults to :attr:`~tecio.DataPacking.BLOCK`.
 
         Raises:
-            NotImplementedError: For FEPOLYGON, FEPOLYHEDRON, or if *datapacking*
-                                 is :attr:`~tecio.libtecio.DataPacking.POINT`.
-            ValueError:          On variable count or array length mismatch; if
-                                 ``node_map`` is omitted without ``con_sharing``; or if
-                                 ``var_sharing``/``con_sharing`` reference a zone with
-                                 no recorded node/cell count, or one whose count
-                                 disagrees with this zone's.
-            ValueError:          If I/O operation attempted on closed or None file
-                                 handle.
-            RuntimeError:        If attempting to write variable aux before variables
-                                 are defined.
-
+            NotImplementedError:
+                For FEPOLYGON or FEPOLYHEDRON.
+            TypeError:
+                If *passive_vars*/*var_sharing* isn't a list/tuple/set/mapping as
+                appropriate, or mixes variable names and indices.
+            ValueError:
+                On variable count or array length mismatch; if ``node_map`` is omitted
+                without ``con_sharing``; if ``var_sharing``/``con_sharing`` reference a
+                zone with no recorded node/cell count, or one whose count disagrees with
+                this zone's; or a variable named in both *passive_vars* and
+                *var_sharing*.
+            ValueError:
+                If I/O operation attempted on closed or None file handle.
+            RuntimeError:
+                If attempting to write variable aux before variables are defined.
+            KeyError:
+                A *passive_vars*/*var_sharing* name doesn't match any dataset variable.
+            IndexError:
+                A *passive_vars*/*var_sharing* index is out of range.
         """
         if isinstance(datapacking, str):
             try:
@@ -687,15 +735,16 @@ class TecplotDatWriter(TecplotWriter):
                 f"Zone type {zone_type!r} is not supported by write_fe_zone."
             )
 
+        new_zone_index = self.current_zone + 1
         if title is None:
-            title = f"FE_Zone_{self.current_zone + 1}"
+            title = f"FE_Zone_{new_zone_index}"
         if variables is None:
             variables = [f"V{i}" for i in range(1, len(data) + 1)]
 
         # Open the file if lazily loaded and flush buffered aux data
         if not self._opened:
             self._open(variables)
-        if self.current_zone == 0:
+        if self.current_zone == -1:
             self.flush_aux()
 
         variable_types = [
@@ -709,7 +758,7 @@ class TecplotDatWriter(TecplotWriter):
         if value_locations is None:
             value_locations = [ValueLocation.NODAL] * len(data)
 
-        prepared: PreparedFEZone = prepare_fe_zone(
+        prepared: PreparedFEZone | None = prepare_fe_zone(
             arrays,
             variable_types,
             zone_type,
@@ -724,8 +773,10 @@ class TecplotDatWriter(TecplotWriter):
             meta=self._meta,
             on_error=self._handle_zone_error,
         )
-        passive_vars = prepared.passive_vars
-        var_sharing = prepared.var_sharing
+        if prepared is None:
+            return
+        passive_flags = prepared.passive_vars
+        share_idx = prepared.var_sharing
         con_sharing = prepared.con_sharing
         num_nodes = prepared.num_nodes
         num_cells = prepared.num_cells
@@ -750,8 +801,8 @@ class TecplotDatWriter(TecplotWriter):
             num_elements=num_cells,
             solution_time=solution_time,
             strand_id=strand_id,
-            passive_vars=passive_vars,
-            var_sharing=var_sharing,
+            passive_vars=passive_flags,
+            var_sharing=share_idx,
             value_locations_global=value_locations_global,
             variable_types_global=variable_types_global,
             con_sharing=con_sharing,
@@ -786,19 +837,26 @@ class TecplotDatWriter(TecplotWriter):
                 cast = np.asarray(arr, dtype=_DT_TO_DTYPE[dt]).ravel()
                 _stage_float_array(buf, cast, self._float_fmt)
 
-        # Write connectivity (if not shared)
-        if not con_sharing:
+        # Write connectivity (if not shared). Node numbers are 0-based in Python;
+        # ASCII text always uses 1-based node numbers.
+        if con_sharing is None:
             assert node_map is not None
-            conn = np.asarray(node_map, dtype=np.intp).reshape(
-                num_cells, _NODES_PER_ELEM[zone_type]
+            conn = (
+                np.asarray(node_map, dtype=np.intp).reshape(
+                    num_cells, _NODES_PER_ELEM[zone_type]
+                )
+                + 1
             )
             for row in conn:
                 _stage_connectivity_row(buf, row)
 
-        # Write face-neighbor connections (if provided)
+        # Write face-neighbor connections (if provided). Cell and remote-zone
+        # references are 0-based in Python; ASCII text is always fully 1-based.
         if face_neighbors_arr is not None:
             assert face_neighbor_mode is not None  # narrowed above
-            flat = face_neighbors_arr
+            flat = convert_face_connections_to_one_based(
+                face_neighbors_arr, face_neighbor_mode
+            )
             one_to_one = (
                 FaceNeighborMode.LOCAL_ONE_TO_ONE,
                 FaceNeighborMode.GLOBAL_ONE_TO_ONE,
@@ -822,7 +880,7 @@ class TecplotDatWriter(TecplotWriter):
                     i += record_len
 
         self._check_handle().write(buf.getvalue())
-        self.current_zone += 1
+        self.current_zone = new_zone_index
 
         # Finally set zone metadata after successfully completing TecIO calls
         self._meta.record_zone(
@@ -836,8 +894,8 @@ class TecplotDatWriter(TecplotWriter):
                 num_nodes=num_nodes,
                 num_elements=num_cells,
                 value_locations=tuple(value_locations_global),
-                passive_vars=tuple(bool(p) for p in passive_vars),
-                shared_vars=tuple(int(s) for s in var_sharing),
+                passive_vars=tuple(bool(p) for p in passive_flags),
+                shared_vars=tuple(share_idx),
                 data_types=tuple(variable_types_global),
                 face_neighbor_mode=(
                     face_neighbor_mode if face_neighbors_arr is not None else None
@@ -850,7 +908,7 @@ class TecplotDatWriter(TecplotWriter):
 
     def _handle_zone_error(self) -> None:
         """Delete the output file if no zone has been committed yet."""
-        if self.current_zone == 0:
+        if self.current_zone == -1:
             if self._fp is not None:
                 self._fp.close()
                 self._fp = None
@@ -880,10 +938,10 @@ class TecplotDatWriter(TecplotWriter):
         solution_time: float = 0.0,
         strand_id: int = 0,
         passive_vars: Sequence[bool | int] | None = None,
-        var_sharing: Sequence[int] | None = None,
+        var_sharing: Sequence[int | None] | None = None,
         value_locations_global: list[ValueLocation] | None = None,
         variable_types_global: list[DataType] | None = None,
-        con_sharing: int = 0,
+        con_sharing: int | None = None,
         aux: dict[str, Any] | None = None,
         datapacking: DataPacking = DataPacking.BLOCK,
         num_face_connections: int = 0,
@@ -940,13 +998,15 @@ class TecplotDatWriter(TecplotWriter):
             if pidx:
                 buf.write(f"{_INDENT}PASSIVEVARLIST=[{','.join(pidx)}]\n")
 
-        if var_sharing and any(var_sharing):
-            entries = [f"[{i + 1}]={z}" for i, z in enumerate(var_sharing) if z]
+        if var_sharing and any(z is not None for z in var_sharing):
+            entries = [
+                f"[{i + 1}]={z + 1}" for i, z in enumerate(var_sharing) if z is not None
+            ]
             if entries:
                 buf.write(f"{_INDENT}VARSHARELIST=({','.join(entries)})\n")
 
-        if con_sharing:
-            buf.write(f"{_INDENT}CONNECTIVITYSHAREZONE={con_sharing}\n")
+        if con_sharing is not None:
+            buf.write(f"{_INDENT}CONNECTIVITYSHAREZONE={con_sharing + 1}\n")
 
         if aux:
             for name, value in aux.items():