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

[serialization] Serialize stub definitions of external parameters. #7926

Merged
merged 2 commits into from
Nov 28, 2023

Conversation

derek-gerstmann
Copy link
Contributor

Add deserialize_parameter methods to allow the user to only deserialize the mapping of external parameters (and remap them to their own user parameters) prior to deserializing the full pipeline definition. Updated tutorial lesson 23.

See also #7849

Add deserialize_parameter methods to allow the user to only deserialize
the mapping of external parameters (and remap them to their own user
parameters) prior to deserializing the full pipeline definition.
@derek-gerstmann derek-gerstmann changed the title Serialize stub definitions of external parameters. [serialization] Serialize stub definitions of external parameters. Oct 30, 2023
Copy link
Contributor

@TH3CHARLie TH3CHARLie left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks for the improvement

@steven-johnson
Copy link
Contributor

Ready to land?

@derek-gerstmann
Copy link
Contributor Author

Yep!

@derek-gerstmann derek-gerstmann merged commit 976ea0b into main Nov 28, 2023
3 checks passed
ardier pushed a commit to ardier/Halide-mutation that referenced this pull request Mar 3, 2024
…alide#7926)

* Serialize stub definitions of external parameters.
Add deserialize_parameter methods to allow the user to only deserialize
the mapping of external parameters (and remap them to their own user
parameters) prior to deserializing the full pipeline definition.

* Clang tidy/format pass

---------

Co-authored-by: Derek Gerstmann <dgerstmann@adobe.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants