Skip to content

Engine: Revert the workaround of increasing maximum stack size#7230

Merged
agoscinski merged 1 commit into
aiidateam:mainfrom
khsrali:recursion
Feb 27, 2026
Merged

Engine: Revert the workaround of increasing maximum stack size#7230
agoscinski merged 1 commit into
aiidateam:mainfrom
khsrali:recursion

Conversation

@khsrali

@khsrali khsrali commented Feb 20, 2026

Copy link
Copy Markdown
Collaborator

Following the changes of #7206 this workaround from #6052 is no longer needed

@codecov

codecov Bot commented Feb 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.70%. Comparing base (e3217c0) to head (4facf16).
⚠️ Report is 104 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7230      +/-   ##
==========================================
- Coverage   79.72%   79.70%   -0.01%     
==========================================
  Files         565      565              
  Lines       43867    43848      -19     
==========================================
- Hits        34967    34945      -22     
- Misses       8900     8903       +3     

☔ 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.

@khsrali khsrali changed the title Recursion Engine: Revert the workaround of increasing maximum stack size Feb 25, 2026
@khsrali khsrali marked this pull request as ready for review February 27, 2026 16:29
@khsrali khsrali requested a review from agoscinski February 27, 2026 16:30
@agoscinski

Copy link
Copy Markdown
Collaborator

We did a test for the recursion limit in this PR aiidateam/plumpy#331 (comment)

@agoscinski agoscinski merged commit 4c6bba9 into aiidateam:main Feb 27, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants