Merge branch 'maint16_3'

# Conflicts:
#	ChangeLog

Author: tturocy

.github/workflows/python.yml

@@ -32,6 +32,52 @@ jobs:
       - name: Run tests
         run: pytest
 
+  macos-13:
+    runs-on: macos-13
+    if: github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name != github.event.pull_request.base.repo.full_name
+    strategy:
+      matrix:
+        python-version: ['3.12']
+
+    steps:
+      - uses: actions/checkout@v4
+      - name: Set up Python ${{ matrix.python-version }}
+        uses: actions/setup-python@v5
+        with:
+          python-version: ${{ matrix.python-version }}
+      - name: Set up dependencies
+        run: |
+          python -m pip install --upgrade pip
+          pip install cython pytest wheel lxml numpy scipy
+      - name: Build extension
+        run: |
+          python -m pip install -v .
+      - name: Run tests
+        run: pytest
+
+  macos-14:
+    runs-on: macos-14
+    if: github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name != github.event.pull_request.base.repo.full_name
+    strategy:
+      matrix:
+        python-version: ['3.12']
+
+    steps:
+      - uses: actions/checkout@v4
+      - name: Set up Python ${{ matrix.python-version }}
+        uses: actions/setup-python@v5
+        with:
+          python-version: ${{ matrix.python-version }}
+      - name: Set up dependencies
+        run: |
+          python -m pip install --upgrade pip
+          pip install cython pytest wheel lxml numpy scipy
+      - name: Build extension
+        run: |
+          python -m pip install -v .
+      - name: Run tests
+        run: pytest
+
   windows:
     runs-on: windows-latest
     if: github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name != github.event.pull_request.base.repo.full_name

ChangeLog

@@ -8,6 +8,13 @@
   been removed as planned. (#357)
 
 
+## [16.3.1] - unreleased
+
+### Fixed
+- Corrected a regression in which information sets were prematurely invalidated (and therefore
+  `delete this` called on them) when removing the last node from an information set.
+
+
 ## [16.3.0] - 2025-01-13
 
 ### General

src/games/gametree.cc

@@ -139,7 +139,6 @@ void GameTreeActionRep::DeleteAction()
     member->m_children[where]->Invalidate();
     erase_atindex(member->m_children, where);
   }
-
   if (m_infoset->IsChanceInfoset()) {
     m_infoset->m_efg->NormalizeChanceProbs(m_infoset);
   }
@@ -263,11 +262,11 @@ void GameTreeInfosetRep::RemoveMember(GameTreeNodeRep *p_node)
   if (m_members.empty()) {
     m_player->m_infosets.erase(
         std::find(m_player->m_infosets.begin(), m_player->m_infosets.end(), this));
-    Invalidate();
     int iset = 1;
     for (auto &infoset : m_player->m_infosets) {
       infoset->m_number = iset++;
     }
+    Invalidate();
   }
 }