Skip to content

Make the recursion limit workaround dependent on the aiida-core version#72

Merged
agoscinski merged 1 commit intoaiidateam:mainfrom
agoscinski:recursion-limit
Mar 1, 2026
Merged

Make the recursion limit workaround dependent on the aiida-core version#72
agoscinski merged 1 commit intoaiidateam:mainfrom
agoscinski:recursion-limit

Conversation

@agoscinski
Copy link
Contributor

With aiida-core v2.8.0 we switched from nest-asyncio to greenlet and therefore do not need to dynamically increase the recursion limit anymore.

@codecov-commenter
Copy link

codecov-commenter commented Mar 1, 2026

Codecov Report

❌ Patch coverage is 86.66667% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.87%. Comparing base (4df7350) to head (36421b7).

Files with missing lines Patch % Lines
src/aiida_pythonjob/decorator.py 86.66% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #72      +/-   ##
==========================================
+ Coverage   89.82%   89.87%   +0.05%     
==========================================
  Files          22       22              
  Lines        1317     1324       +7     
==========================================
+ Hits         1183     1190       +7     
  Misses        134      134              

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

With aiida-core v2.8.0 we switched from nest-asyncio to greenlet and
therefore do not need to dynamically increase the recursion limit
anymore.
@agoscinski agoscinski merged commit 86f21bd into aiidateam:main Mar 1, 2026
6 checks passed
@agoscinski agoscinski deleted the recursion-limit branch March 1, 2026 19:14
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