Skip to content

Replace Boost.python with Pybind11 ECFLOW-2090#269

Open
marcosbento wants to merge 15 commits into
developfrom
feature/pybind11
Open

Replace Boost.python with Pybind11 ECFLOW-2090#269
marcosbento wants to merge 15 commits into
developfrom
feature/pybind11

Conversation

@marcosbento
Copy link
Copy Markdown
Collaborator

@marcosbento marcosbento commented Jan 23, 2026

Description

Replace Boost.python with pybind11

Contributor Declaration

By opening this pull request, I affirm the following:

  • All authors agree to the Contributor License Agreement.
  • The code follows the project's coding standards.
  • I have performed self-review and added comments where needed.
  • I have added or updated tests to verify that my changes are effective and functional.
  • I have run all existing tests and confirmed they pass.

@marcosbento marcosbento marked this pull request as draft January 23, 2026 15:38
@marcosbento marcosbento force-pushed the feature/pybind11 branch 4 times, most recently from 326501c to 2fd6d48 Compare February 4, 2026 09:53
@marcosbento marcosbento self-assigned this Feb 5, 2026
@marcosbento marcosbento force-pushed the feature/pybind11 branch 5 times, most recently from 89c3732 to 0a5e39f Compare February 11, 2026 11:28
@marcosbento marcosbento force-pushed the feature/pybind11 branch 5 times, most recently from 2ed61fa to 7778372 Compare February 20, 2026 09:52
@marcosbento marcosbento force-pushed the feature/pybind11 branch 2 times, most recently from 332418e to 226ec98 Compare March 3, 2026 15:59
@marcosbento marcosbento force-pushed the feature/pybind11 branch 2 times, most recently from 11b8c42 to ca069b4 Compare March 6, 2026 08:52
@marcosbento marcosbento force-pushed the feature/pybind11 branch 2 times, most recently from 558e90b to 5e203ee Compare March 17, 2026 12:59
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 18, 2026 •
edited
Loading

Codecov Report

❌ Patch coverage is 93.13131% with 68 lines in your changes missing coverage. Please review.
✅ Project coverage is 50.10%. Comparing base (dd96e47) to head (9b4871a).

Files with missing lines Patch % Lines
libs/pyext/src/ecflow/python/ExportClient.cpp 88.93% 27 Missing ⚠️
libs/pyext/src/ecflow/python/ExportNode.cpp 95.49% 14 Missing ⚠️
libs/pyext/src/ecflow/python/NodeUtil.cpp 89.10% 11 Missing ⚠️
libs/pyext/src/ecflow/python/ExportDefs.cpp 96.22% 4 Missing ⚠️
libs/pyext/src/ecflow/python/PythonUtil.cpp 85.71% 4 Missing ⚠️
libs/pyext/src/ecflow/python/PythonBinding.hpp 76.92% 3 Missing ⚠️
libs/pyext/src/ecflow/python/NodeAttrDoc.cpp 80.00% 2 Missing ⚠️
libs/pyext/src/ecflow/python/Trigger.cpp 66.66% 2 Missing ⚠️
libs/pyext/src/ecflow/python/ExportTask.cpp 96.42% 1 Missing ⚠️
Additional details and impacted files 
@@             Coverage Diff             @@
##           develop     #269      +/-   ##
===========================================
+ Coverage    49.96%   50.10%   +0.14%     
===========================================
  Files         1238     1240       +2     
  Lines        98169    98345     +176     
  Branches     14950    14933      -17     
===========================================
+ Hits         49049    49278     +229     
+ Misses       49120    49067      -53

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@marcosbento marcosbento changed the title Replace Boost.python with pybind11 Replace Boost.python with Pybind11 ECFLOW-2090 May 11, 2026
@marcosbento marcosbento marked this pull request as ready for review May 12, 2026 14:56
@marcosbento marcosbento force-pushed the feature/pybind11 branch 10 times, most recently from e13476f to 599ff28 Compare May 16, 2026 12:23
marcosbento added 15 commits May 16, 2026 14:02
@marcosbento
Increase unit test coverage of all python bindings
f9a899d 
Increase the test coverage, and thus confidence, for the upcoming
change from Boost.python to pybind11.

Re ECFLOW-2090
@marcosbento
Replace Boost.python with Pybind11
cd20273 
Re ECFLOW-2090
@marcosbento
clean-up ExportClient
97bde97 
Re ECFLOW-2090
@marcosbento
clean-up ExportNode
6976652 
Re ECFLOW-2090
@marcosbento
clean-up ExportNodeAttr
8cdf6ce 
Re ECFLOW-2090
@marcosbento
clean-up ExportSuiteAndFamily
0183a5d 
Re ECFLOW-2090
@marcosbento
clean-up ExportTask
9ae6785 
Re ECFLOW-2090
@marcosbento
clean-up ExportDefs
a005b1a 
Re ECFLOW-2090
@marcosbento
Add ecflow.__version__ information
c20a65e 
Re ECFLOW-2090
@marcosbento
Configure HPC CI to find pybind11
2ee90bb 
Re ECFLOW-2090
@marcosbento
Make direct use as string of dict/kwargs key
005077f 
A dict/kwargs key is guaranteed to be string by construction, so it is
unnecessary to attempt extraction with error handling. Direct casting
the object can be used instead.

Re ECFLOW-2090
@marcosbento
Make extraction mechanism consistent for py::handle/py::obj
1d27235 
Re ECFLOW-2090
@marcosbento
Add documentation to the objects binding functions
664bc9c 
Re ECFLOW-2090
@marcosbento
Correct issues identified during review
79e8d26 
Re ECFLOW-2090
@marcosbento
Adapt documentation scripts to pybind11
bd6dc74 
Re ECFLOW-2090
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@marcosbento marcosbento

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@marcosbento @codecov-commenter