Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Map returns cached outputs if the inputs are unchanged #1709

Merged
merged 1 commit into from
Jun 8, 2024

Conversation

alecthomas
Copy link
Collaborator

This incurs a slight penalty due to the JSON serialisation + hashing, but it should be relatively insignificant in most cases.

cc @mistermoe

@alecthomas alecthomas requested review from a team and matt2e and removed request for a team June 7, 2024 22:28
@ftl-robot ftl-robot mentioned this pull request Jun 7, 2024
@alecthomas alecthomas force-pushed the aat/map-cache branch 2 times, most recently from 8b004a2 to 7700a43 Compare June 7, 2024 23:56
This incurs a slight penalty due to the JSON serialisation + hashing,
but it should be relatively insignificant in most cases.
@alecthomas alecthomas merged commit 520cf6d into main Jun 8, 2024
38 checks passed
@alecthomas alecthomas deleted the aat/map-cache branch June 8, 2024 00:34
@matt2e matt2e added the approved Marks an already closed PR as approved label Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Marks an already closed PR as approved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants