Skip to content

Add GiveUniqueParameterTensors to ModelWrapper.cleanup transformations#227

Closed
iksnagreb wants to merge 1 commit intofastmachinelearning:mainfrom
iksnagreb:fix/unique-parameters
Closed

Add GiveUniqueParameterTensors to ModelWrapper.cleanup transformations#227
iksnagreb wants to merge 1 commit intofastmachinelearning:mainfrom
iksnagreb:fix/unique-parameters

Conversation

@iksnagreb
Copy link
Collaborator

Fixes transformations which create copies of nodes without copying parameter tensors, resulting in unwanted entanglement. This manifests as later transformations having non-local effects, manipulating seemingly unrelated nodes.

Fixes transformations which create copies of nodes without copying
parameter tensors, resulting in unwanted entanglement. This manifests as
later transformations having non-local effects, manipulating seemingly
unrelated nodes.
@iksnagreb iksnagreb added this to the v1.0.0 milestone Jan 23, 2026
Copy link
Collaborator

@makoeppel makoeppel left a comment

Choose a reason for hiding this comment

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

Is this not the same as: #168

@iksnagreb
Copy link
Collaborator Author

Is this not the same as: #168

Oh, yes...

@iksnagreb iksnagreb closed this Jan 23, 2026
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.

2 participants