Skip to content

feat: support nested tuples of arrays as Caches#748

Merged
gdalle merged 15 commits intomainfrom
gd/recsim
Mar 18, 2025
Merged

feat: support nested tuples of arrays as Caches#748
gdalle merged 15 commits intomainfrom
gd/recsim

Conversation

@gdalle
Copy link
Copy Markdown
Member

@gdalle gdalle commented Mar 18, 2025

Fixes #747

  • Implement recursive_similar for nested (named) tuples of arrays
  • Use or replicate it in every operator overloading backend when handling caches
  • Modify cachified test scenarios so that they run on a named tuple by default

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 18, 2025

Codecov Report

Attention: Patch coverage is 90.24390% with 4 lines in your changes missing coverage. Please review.

Project coverage is 97.84%. Comparing base (e1d171f) to head (8a0c9fd).

Files with missing lines Patch % Lines
...Ext/DifferentiationInterfaceTestStaticArraysExt.jl 33.33% 2 Missing ⚠️
...fferentiationInterfaceTest/src/scenarios/modify.jl 84.61% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #748      +/-   ##
==========================================
- Coverage   97.89%   97.84%   -0.05%     
==========================================
  Files         128      128              
  Lines        7368     7383      +15     
==========================================
+ Hits         7213     7224      +11     
- Misses        155      159       +4     
Flag Coverage Δ
DI 98.94% <100.00%> (+<0.01%) ⬆️
DIT 95.07% <80.00%> (-0.17%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@gdalle gdalle marked this pull request as ready for review March 18, 2025 15:06
@gdalle gdalle added the skipci Don't run CI tests and docs label Mar 18, 2025
@gdalle gdalle merged commit 7d252ad into main Mar 18, 2025
4 checks passed
@gdalle gdalle deleted the gd/recsim branch May 12, 2025 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skipci Don't run CI tests and docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support more complex caches

1 participant