Skip to content

feat: allow computing derived resources via evaluate function#288

Merged
dexter2206 merged 16 commits into
mainfrom
ast-3539-add-derived-resources
May 20, 2026
Merged

feat: allow computing derived resources via evaluate function#288
dexter2206 merged 16 commits into
mainfrom
ast-3539-add-derived-resources

Conversation

@mstechly
Copy link
Copy Markdown
Contributor

@mstechly mstechly commented Apr 22, 2026

Description

Adds possibility to add derived resources during routine evaluation (as opposed to only during compilation, as we do now). This change is being made to facilitate unified resources tracking elsewhere in our software stack. To this end, all machinery related to derived resources has been moved from _compile module to _common module in the same subpackage.

Additionally, this also includes following boy-scout changes:

  • Adds empty lines at the top of so me modules (added by original author @mstechly )
  • Updates format in pre-commit config (we were using deprecated hook names)
  • Gets rid of some scipy (we were passing deprecated options to optimizers)
  • Fixes typo (for whatever reason we called a single derived resource definition DerivedResources instead of DerivedResource).

Please verify that you have completed the following steps

  • I have self-reviewed my code.
  • I have included test cases validating introduced feature/fix.
  • I have updated documentation.

@mstechly mstechly requested a review from dexter2206 April 22, 2026 08:26
@cla-bot cla-bot Bot added the cla-signed label Apr 22, 2026
@mstechly mstechly changed the title Ast 3539 add derived resources feat: add derived resources Apr 23, 2026
@dexter2206 dexter2206 self-assigned this May 20, 2026
@dexter2206 dexter2206 changed the title feat: add derived resources feat: allow computing derived resources via evaluate function May 20, 2026
@dexter2206 dexter2206 merged commit d66f2a3 into main May 20, 2026
9 checks passed
@dexter2206 dexter2206 deleted the ast-3539-add-derived-resources branch May 20, 2026 16:37
This was referenced May 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants