Skip to content

Replace posts with ToDense layer implementation - #5418

Open
MaddipatlaChetan24 wants to merge 1 commit into
uber:mainfrom
MaddipatlaChetan24:patch-6
Open

MaddipatlaChetan24 wants to merge 1 commit into
uber:mainfrom
MaddipatlaChetan24:patch-6

Conversation

@MaddipatlaChetan24

Copy link
Copy Markdown
Contributor

Removed the existing posts array and added a new ToDense layer implementation for handling composite tensors in Keras models.

Description

Simplifies ToDense.get_config() to merge base_config and config using a dict-unpacking merge ({**base_config, **config}) instead of converting both to lists, concatenating them, and re-wrapping with dict(...). Verified behaviorally identical, including on the key-overlap case (config's values still take precedence over base_config's, matching the original's + ordering) — same output dict either way, just without the two throwaway list allocations. No other changes.

Scope

Patch: Cleanup

Removed the existing posts array and added a new ToDense layer implementation for handling composite tensors in Keras models.
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