Safe-DS / Runner

Execute Safe-DS programs that were compiled to Python.
MIT License
2 stars 0 forks source link

feat: track memoization stats #51

Closed WinPlay02 closed 8 months ago

WinPlay02 commented 8 months ago

Closes partially #44

Summary of Changes

github-actions[bot] commented 8 months ago

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ PYTHON black 3 0 0 0.77s
✅ PYTHON mypy 3 0 2.29s
✅ PYTHON ruff 3 0 0 0.03s
✅ REPOSITORY git_diff yes no 0.01s

See detailed report in MegaLinter reports _Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff_

_MegaLinter is graciously provided by OX Security_

codecov[bot] commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (9ee959d) 100.00% compared to head (b501fce) 100.00%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #51 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 9 10 +1 Lines 436 499 +63 ========================================= + Hits 436 499 +63 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

WinPlay02 commented 8 months ago

I would propose to merge the tracking of stats (this PR) first and continue the strategies for memoizing and removing in a follow-up PR.

lars-reimann commented 8 months ago

I would propose to merge the tracking of stats (this PR) first and continue the strategies for memoizing and removing in a follow-up PR.

I'm all for that.

WinPlay02 commented 8 months ago

I've made some changes that were easier to implement directly than to describe them.

@WinPlay02 Please take a quick look whether I broke anything and then feel free to merge if you're happy with it (or complain if you are not).

This should be good to go

lars-reimann commented 8 months ago

:tada: This PR is included in version 0.6.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: