Skip to content
This repository was archived by the owner on Dec 9, 2025. It is now read-only.

Some memo refactors#108

Merged
AlSchlo merged 3 commits into
mainfrom
connor/clean-memo
May 14, 2025
Merged

Some memo refactors#108
AlSchlo merged 3 commits into
mainfrom
connor/clean-memo

Conversation

@connortsui20

Copy link
Copy Markdown
Member

Problem

There were a few inefficiencies in the memo table (mainly extraneous clones, and how remapping of objects like goals and expressions work). Also, some of the function docs were quite verbose.

Summary of changes

This PR cleans up some of the memo table modules style and documentation, as well as removes several extraneous clones.

@connortsui20 connortsui20 requested a review from AlSchlo May 14, 2025 14:30
Comment thread optd/src/memo/memory/helpers.rs
@connortsui20 connortsui20 changed the title Connor/clean memo Some memo refactors May 14, 2025
@codecov-commenter

codecov-commenter commented May 14, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 70.53571% with 33 lines in your changes missing coverage. Please review.

Project coverage is 82.6%. Comparing base (ed681b5) to head (f51e4f3).
Report is 7 commits behind head on main.

Files with missing lines Patch % Lines
optd/src/memo/memory/materialize.rs 27.5% 21 Missing ⚠️
optd/src/memo/memory/helpers.rs 85.5% 12 Missing ⚠️
Additional details and impacted files
Files with missing lines Coverage Δ
optd/src/memo/memory/implementation.rs 90.8% <ø> (ø)
optd/src/memo/memory/helpers.rs 85.9% <85.5%> (-2.1%) ⬇️
optd/src/memo/memory/materialize.rs 58.4% <27.5%> (+1.1%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@AlSchlo AlSchlo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks all good to me, please ensure that nothing gets broken by running the fuzzing tests again.

@AlSchlo AlSchlo merged commit 570efc9 into main May 14, 2025
12 checks passed
@AlSchlo AlSchlo deleted the connor/clean-memo branch May 14, 2025 20:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants