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

Allow Sources to accept multiple model functions. #618

Merged
merged 1 commit into from
Dec 24, 2024

Conversation

copybara-service[bot]
Copy link

@copybara-service copybara-service bot commented Dec 18, 2024

Allow Sources to accept multiple model functions.

This modifies the sources registry to register a dict of model functions. These will be tied to runtime params and either autocreate a builder (or a given source builder class can be provided that will be used)

Follow up changes to:

  • move all functions to be used through model_func.
  • remove function and affected_core_profiles.
  • add a Protocol for SourceProfileFunctions that will define the interface.
  • open the registry to accept new model functions.

@copybara-service copybara-service bot force-pushed the test_706769008 branch 3 times, most recently from afa948f to af68785 Compare December 18, 2024 15:08
@copybara-service copybara-service bot changed the title Allow for flexible model functions for Sources. Allow Sources to accept multiple model functions. Dec 18, 2024
@copybara-service copybara-service bot force-pushed the test_706769008 branch 5 times, most recently from 237128a to 18f5d0a Compare December 24, 2024 12:22
This modifies the sources registry to register a dict of model functions. These will be tied to runtime params and either autocreate a builder (or a given source builder class can be provided that will be used)

Follow up changes to:
- move all `function`s to be used through `model_func`.
- remove `function` and `affected_core_profiles`.
- add a `Protocol` for `SourceProfileFunctions` that will define the interface.
- open the registry to accept new model functions.

PiperOrigin-RevId: 709309449
@copybara-service copybara-service bot merged commit 0ccedf9 into main Dec 24, 2024
@copybara-service copybara-service bot deleted the test_706769008 branch December 24, 2024 12:27
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.

1 participant