Skip to content

Flatten __call__ and simplify _combine in CombinedTransportModel. - #2362

Open
copybara-service[bot] wants to merge 1 commit into
mainfrom
test_964556808
Open

Flatten __call__ and simplify _combine in CombinedTransportModel.#2362
copybara-service[bot] wants to merge 1 commit into
mainfrom
test_964556808

Conversation

@copybara-service

Copy link
Copy Markdown

Flatten call and simplify _combine in CombinedTransportModel.

  • Merge call_implementation() body into call(), removing the separate method. call now directly combines core and pedestal coefficients, applies clipping, and applies smoothing.
  • Simplify _combine loop: call model() directly instead of separate model.call_implementation() + model.zero_out_disabled_channels(), since ComponentTransportModel.call now wraps both.
  • Update combined_test.py to call model() instead of model.call_implementation().

- Merge call_implementation() body into __call__(), removing the separate method. __call__ now directly combines core and pedestal coefficients, applies clipping, and applies smoothing.
- Simplify _combine loop: call model() directly instead of separate model.call_implementation() + model.zero_out_disabled_channels(), since ComponentTransportModel.__call__ now wraps both.
- Update combined_test.py to call model() instead of model.call_implementation().

PiperOrigin-RevId: 964556808
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