Skip to content

Commit f401a90

Browse files
authored
Merge pull request #2060 from borglab/fan/fix_pybind_version
Fix incorrect pybind version in `wrap`
2 parents 528c2ee + b332412 commit f401a90

File tree

190 files changed

+1429
-11131
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

190 files changed

+1429
-11131
lines changed

gtsam/base/Lie.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -313,7 +313,7 @@ class IsMatrixLieGroup: public IsLieGroup<T> {
313313
typedef typename traits<T>::LieAlgebra LieAlgebra;
314314
typedef typename traits<T>::TangentVector TangentVector;
315315

316-
BOOST_CONCEPT_USAGE(IsMatrixLieGroup) {
316+
GTSAM_CONCEPT_USAGE(IsMatrixLieGroup) {
317317
// hat and vee
318318
X = traits<T>::Hat(xi);
319319
xi = traits<T>::Vee(X);

wrap/pybind11/.codespell-ignore-lines

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -12,17 +12,6 @@ template <op_id id, op_type ot, typename L, typename R>
1212
template <detail::op_id id, detail::op_type ot, typename L, typename R, typename... Extra>
1313
class_ &def(const detail::op_<id, ot, L, R> &op, const Extra &...extra) {
1414
class_ &def_cast(const detail::op_<id, ot, L, R> &op, const Extra &...extra) {
15-
int valu;
16-
explicit movable_int(int v) : valu{v} {}
17-
movable_int(movable_int &&other) noexcept : valu(other.valu) { other.valu = 91; }
18-
explicit indestructible_int(int v) : valu{v} {}
19-
REQUIRE(hld.as_raw_ptr_unowned<zombie>()->valu == 19);
20-
REQUIRE(othr.valu == 19);
21-
REQUIRE(orig.valu == 91);
22-
(m.pass_valu, "Valu", "pass_valu:Valu(_MvCtor)*_CpCtor"),
23-
atyp_valu rtrn_valu() { atyp_valu obj{"Valu"}; return obj; }
24-
assert m.atyp_valu().get_mtxt() == "Valu"
25-
// valu(e), ref(erence), ptr or p (pointer), r = rvalue, m = mutable, c = const,
2615
@pytest.mark.parametrize("access", ["ro", "rw", "static_ro", "static_rw"])
2716
struct IntStruct {
2817
explicit IntStruct(int v) : value(v){};

wrap/pybind11/.github/CONTRIBUTING.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ nox -s build
8181
### Full setup
8282

8383
To setup an ideal development environment, run the following commands on a
84-
system with CMake 3.15+:
84+
system with CMake 3.14+:
8585

8686
```bash
8787
python3 -m venv venv
@@ -96,8 +96,8 @@ Tips:
9696
* You can use `virtualenv` (faster, from PyPI) instead of `venv`.
9797
* You can select any name for your environment folder; if it contains "env" it
9898
will be ignored by git.
99-
* If you don't have CMake 3.15+, just add "cmake" to the pip install command.
100-
* You can use `-DPYBIND11_FINDPYTHON=ON` to use FindPython.
99+
* If you don't have CMake 3.14+, just add "cmake" to the pip install command.
100+
* You can use `-DPYBIND11_FINDPYTHON=ON` to use FindPython on CMake 3.12+
101101
* In classic mode, you may need to set `-DPYTHON_EXECUTABLE=/path/to/python`.
102102
FindPython uses `-DPython_ROOT_DIR=/path/to` or
103103
`-DPython_EXECUTABLE=/path/to/python`.
@@ -149,8 +149,8 @@ To run the tests, you can "build" the check target:
149149
cmake --build build --target check
150150
```
151151

152-
`--target` can be spelled `-t`. You can also run individual tests with these
153-
targets:
152+
`--target` can be spelled `-t` in CMake 3.15+. You can also run individual
153+
tests with these targets:
154154

155155
* `pytest`: Python tests only, using the
156156
[pytest](https://docs.pytest.org/en/stable/) framework

wrap/pybind11/.github/workflows/ci.yml

Lines changed: 23 additions & 45 deletions
Original file line numberDiff line numberDiff line change
@@ -39,8 +39,6 @@ jobs:
3939
- 'pypy-3.8'
4040
- 'pypy-3.9'
4141
- 'pypy-3.10'
42-
- 'pypy-3.11'
43-
- 'graalpy-24.1'
4442

4543
# Items in here will either be added to the build matrix (if not
4644
# present), or add new keys to an existing matrix element if all the
@@ -66,45 +64,9 @@ jobs:
6664
# Inject a couple Windows 2019 runs
6765
- runs-on: windows-2019
6866
python: '3.9'
69-
# Inject a few runs with different runtime libraries
70-
- runs-on: windows-2022
71-
python: '3.9'
72-
args: >
73-
-DCMAKE_MSVC_RUNTIME_LIBRARY=MultiThreaded
74-
- runs-on: windows-2022
75-
python: '3.10'
76-
args: >
77-
-DCMAKE_MSVC_RUNTIME_LIBRARY=MultiThreadedDLL
78-
# This needs a python built with MTd
79-
# - runs-on: windows-2022
80-
# python: '3.11'
81-
# args: >
82-
# -DCMAKE_MSVC_RUNTIME_LIBRARY=MultiThreadedDebug
83-
- runs-on: windows-2022
84-
python: '3.12'
85-
args: >
86-
-DCMAKE_MSVC_RUNTIME_LIBRARY=MultiThreadedDebugDLL
8767
# Extra ubuntu latest job
8868
- runs-on: ubuntu-latest
8969
python: '3.11'
90-
# Run tests with py::smart_holder as the default holder
91-
# with recent (or ideally latest) released Python version.
92-
- runs-on: ubuntu-latest
93-
python: '3.12'
94-
args: >
95-
-DCMAKE_CXX_FLAGS="-DPYBIND11_RUN_TESTING_WITH_SMART_HOLDER_AS_DEFAULT_BUT_NEVER_USE_IN_PRODUCTION_PLEASE"
96-
- runs-on: macos-13
97-
python: '3.12'
98-
args: >
99-
-DCMAKE_CXX_FLAGS="-DPYBIND11_RUN_TESTING_WITH_SMART_HOLDER_AS_DEFAULT_BUT_NEVER_USE_IN_PRODUCTION_PLEASE"
100-
- runs-on: windows-2022
101-
python: '3.12'
102-
args: >
103-
-DCMAKE_CXX_FLAGS="/DPYBIND11_RUN_TESTING_WITH_SMART_HOLDER_AS_DEFAULT_BUT_NEVER_USE_IN_PRODUCTION_PLEASE /GR /EHsc"
104-
exclude:
105-
# The setup-python action currently doesn't have graalpy for windows
106-
- python: 'graalpy-24.1'
107-
runs-on: 'windows-2022'
10870

10971

11072
name: "🐍 ${{ matrix.python }} • ${{ matrix.runs-on }} • x64 ${{ matrix.args }}"
@@ -160,7 +122,6 @@ jobs:
160122
-DPYBIND11_DISABLE_HANDLE_TYPE_NAME_DEFAULT_IMPLEMENTATION=ON
161123
-DPYBIND11_SIMPLE_GIL_MANAGEMENT=ON
162124
-DPYBIND11_NUMPY_1_ONLY=ON
163-
-DPYBIND11_PYTEST_ARGS=-v
164125
-DDOWNLOAD_CATCH=ON
165126
-DDOWNLOAD_EIGEN=ON
166127
-DCMAKE_CXX_STANDARD=11
@@ -190,7 +151,6 @@ jobs:
190151
-DPYBIND11_WERROR=ON
191152
-DPYBIND11_SIMPLE_GIL_MANAGEMENT=OFF
192153
-DPYBIND11_NUMPY_1_ONLY=ON
193-
-DPYBIND11_PYTEST_ARGS=-v
194154
-DDOWNLOAD_CATCH=ON
195155
-DDOWNLOAD_EIGEN=ON
196156
-DCMAKE_CXX_STANDARD=17
@@ -210,7 +170,6 @@ jobs:
210170
run: >
211171
cmake -S . -B build3
212172
-DPYBIND11_WERROR=ON
213-
-DPYBIND11_PYTEST_ARGS=-v
214173
-DDOWNLOAD_CATCH=ON
215174
-DDOWNLOAD_EIGEN=ON
216175
-DCMAKE_CXX_STANDARD=17
@@ -351,11 +310,22 @@ jobs:
351310
strategy:
352311
fail-fast: false
353312
matrix:
313+
clang:
314+
- 3.6
315+
- 3.7
316+
- 3.9
317+
- 7
318+
- 9
319+
- dev
320+
std:
321+
- 11
354322
container_suffix:
355323
- ""
356324
include:
357325
- clang: 5
358326
std: 14
327+
- clang: 10
328+
std: 17
359329
- clang: 11
360330
std: 20
361331
- clang: 12
@@ -533,6 +503,10 @@ jobs:
533503
fail-fast: false
534504
matrix:
535505
include:
506+
- { gcc: 7, std: 11 }
507+
- { gcc: 7, std: 17 }
508+
- { gcc: 8, std: 14 }
509+
- { gcc: 8, std: 17 }
536510
- { gcc: 9, std: 20 }
537511
- { gcc: 10, std: 17 }
538512
- { gcc: 10, std: 20 }
@@ -753,9 +727,9 @@ jobs:
753727

754728
# This tests an "install" with the CMake tools
755729
install-classic:
756-
name: "🐍 3.9 • Debian • x86 • Install"
730+
name: "🐍 3.7 • Debian • x86 • Install"
757731
runs-on: ubuntu-latest
758-
container: i386/debian:bullseye
732+
container: i386/debian:buster
759733

760734
steps:
761735
- uses: actions/checkout@v1 # v1 is required to run inside docker
@@ -835,6 +809,7 @@ jobs:
835809
fail-fast: false
836810
matrix:
837811
python:
812+
- '3.7'
838813
- '3.8'
839814
- '3.9'
840815
- '3.10'
@@ -852,6 +827,8 @@ jobs:
852827
args: -DCMAKE_CXX_STANDARD=20
853828
- python: '3.8'
854829
args: -DCMAKE_CXX_STANDARD=17
830+
- python: '3.7'
831+
args: -DCMAKE_CXX_STANDARD=14
855832

856833

857834
name: "🐍 ${{ matrix.python }} • MSVC 2019 • x86 ${{ matrix.args }}"
@@ -1030,6 +1007,7 @@ jobs:
10301007
git
10311008
mingw-w64-${{matrix.env}}-gcc
10321009
mingw-w64-${{matrix.env}}-python-pip
1010+
mingw-w64-${{matrix.env}}-python-numpy
10331011
mingw-w64-${{matrix.env}}-cmake
10341012
mingw-w64-${{matrix.env}}-make
10351013
mingw-w64-${{matrix.env}}-python-pytest
@@ -1041,7 +1019,7 @@ jobs:
10411019
with:
10421020
msystem: ${{matrix.sys}}
10431021
install: >-
1044-
mingw-w64-${{matrix.env}}-python-numpy
1022+
git
10451023
mingw-w64-${{matrix.env}}-python-scipy
10461024
mingw-w64-${{matrix.env}}-eigen3
10471025
@@ -1139,7 +1117,7 @@ jobs:
11391117
uses: jwlawson/actions-setup-cmake@v2.0
11401118

11411119
- name: Install ninja-build tool
1142-
uses: seanmiddleditch/gha-setup-ninja@v6
1120+
uses: seanmiddleditch/gha-setup-ninja@v5
11431121

11441122
- name: Run pip installs
11451123
run: |

wrap/pybind11/.github/workflows/configure.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -31,30 +31,30 @@ jobs:
3131
include:
3232
- runs-on: ubuntu-20.04
3333
arch: x64
34-
cmake: "3.15"
34+
cmake: "3.5"
3535

3636
- runs-on: ubuntu-20.04
3737
arch: x64
3838
cmake: "3.29"
3939

4040
- runs-on: macos-13
4141
arch: x64
42-
cmake: "3.15"
42+
cmake: "3.7"
4343

4444
- runs-on: windows-2019
4545
arch: x64 # x86 compilers seem to be missing on 2019 image
4646
cmake: "3.18"
4747

48-
name: 🐍 3.8 • CMake ${{ matrix.cmake }} • ${{ matrix.runs-on }}
48+
name: 🐍 3.7 • CMake ${{ matrix.cmake }} • ${{ matrix.runs-on }}
4949
runs-on: ${{ matrix.runs-on }}
5050

5151
steps:
5252
- uses: actions/checkout@v4
5353

54-
- name: Setup Python 3.8
54+
- name: Setup Python 3.7
5555
uses: actions/setup-python@v5
5656
with:
57-
python-version: 3.8
57+
python-version: 3.7
5858
architecture: ${{ matrix.arch }}
5959

6060
- name: Prepare env

wrap/pybind11/.github/workflows/emscripten.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ jobs:
2222
submodules: true
2323
fetch-depth: 0
2424

25-
- uses: pypa/cibuildwheel@v2.23
25+
- uses: pypa/cibuildwheel@v2.20
2626
env:
2727
PYODIDE_BUILD_EXPORTS: whole_archive
2828
with:

wrap/pybind11/.github/workflows/pip.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ jobs:
103103
- uses: actions/download-artifact@v4
104104

105105
- name: Generate artifact attestation for sdist and wheel
106-
uses: actions/attest-build-provenance@bd77c077858b8d561b7a36cbe48ef4cc642ca39d # v2.2.2
106+
uses: actions/attest-build-provenance@1c608d11d69870c2092266b3f9a6f3abbf17002c # v1.4.3
107107
with:
108108
subject-path: "*/pybind11*"
109109

wrap/pybind11/.pre-commit-config.yaml

Lines changed: 10 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -25,22 +25,22 @@ repos:
2525

2626
# Clang format the codebase automatically
2727
- repo: https://github.com/pre-commit/mirrors-clang-format
28-
rev: "v19.1.7"
28+
rev: "v18.1.8"
2929
hooks:
3030
- id: clang-format
3131
types_or: [c++, c, cuda]
3232

3333
# Ruff, the Python auto-correcting linter/formatter written in Rust
3434
- repo: https://github.com/astral-sh/ruff-pre-commit
35-
rev: v0.9.9
35+
rev: v0.6.3
3636
hooks:
3737
- id: ruff
3838
args: ["--fix", "--show-fixes"]
3939
- id: ruff-format
4040

4141
# Check static types with mypy
4242
- repo: https://github.com/pre-commit/mirrors-mypy
43-
rev: "v1.15.0"
43+
rev: "v1.11.2"
4444
hooks:
4545
- id: mypy
4646
args: []
@@ -62,7 +62,7 @@ repos:
6262

6363
# Standard hooks
6464
- repo: https://github.com/pre-commit/pre-commit-hooks
65-
rev: "v5.0.0"
65+
rev: "v4.6.0"
6666
hooks:
6767
- id: check-added-large-files
6868
- id: check-case-conflict
@@ -76,11 +76,10 @@ repos:
7676
- id: mixed-line-ending
7777
- id: requirements-txt-fixer
7878
- id: trailing-whitespace
79-
exclude: \.patch?$
8079

8180
# Also code format the docs
8281
- repo: https://github.com/adamchainz/blacken-docs
83-
rev: "1.19.1"
82+
rev: "1.18.0"
8483
hooks:
8584
- id: blacken-docs
8685
additional_dependencies:
@@ -91,11 +90,10 @@ repos:
9190
rev: "v1.5.5"
9291
hooks:
9392
- id: remove-tabs
94-
exclude: (^docs/.*|\.patch)?$
9593

9694
# Avoid directional quotes
9795
- repo: https://github.com/sirosen/texthooks
98-
rev: "0.6.8"
96+
rev: "0.6.7"
9997
hooks:
10098
- id: fix-ligatures
10199
- id: fix-smartquotes
@@ -110,7 +108,7 @@ repos:
110108

111109
# Checks the manifest for missing files (native support)
112110
- repo: https://github.com/mgedmin/check-manifest
113-
rev: "0.50"
111+
rev: "0.49"
114112
hooks:
115113
- id: check-manifest
116114
# This is a slow hook, so only run this if --hook-stage manual is passed
@@ -121,7 +119,7 @@ repos:
121119
# Use tools/codespell_ignore_lines_from_errors.py
122120
# to rebuild .codespell-ignore-lines
123121
- repo: https://github.com/codespell-project/codespell
124-
rev: "v2.4.1"
122+
rev: "v2.3.0"
125123
hooks:
126124
- id: codespell
127125
exclude: ".supp$"
@@ -144,14 +142,14 @@ repos:
144142

145143
# PyLint has native support - not always usable, but works for us
146144
- repo: https://github.com/PyCQA/pylint
147-
rev: "v3.3.4"
145+
rev: "v3.2.7"
148146
hooks:
149147
- id: pylint
150148
files: ^pybind11
151149

152150
# Check schemas on some of our YAML files
153151
- repo: https://github.com/python-jsonschema/check-jsonschema
154-
rev: 0.31.2
152+
rev: 0.29.2
155153
hooks:
156154
- id: check-readthedocs
157155
- id: check-github-workflows

0 commit comments

Comments
 (0)