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

fix(transform): order scope layers to prevent namespace pollution #3

Conversation

blepabyte
Copy link
Owner

fix(transform): order scope layers to prevent namespace pollution

Fixes bug where np.load errors with "numpy.AxisError: axis 6 is out of bounds for array of dimension 0" because min was dispatching to np.min due to prior @set_module decorator.

fix: memory leak

@blepabyte blepabyte force-pushed the 06-10-feat_assign_transformed_functions_a_uuid branch from 911fadc to 4cd61ed Compare June 17, 2024 07:52
@blepabyte blepabyte force-pushed the 06-17-fix_transform_order_scope_layers_to_prevent_namespace_pollution branch from eac440a to fe656f8 Compare June 17, 2024 07:52
@blepabyte blepabyte marked this pull request as ready for review June 17, 2024 08:04
@blepabyte blepabyte force-pushed the 06-10-feat_assign_transformed_functions_a_uuid branch from 4cd61ed to 9557d68 Compare June 17, 2024 08:13
@blepabyte blepabyte changed the base branch from 06-10-feat_assign_transformed_functions_a_uuid to master June 17, 2024 08:15
Fixes bug where np.load errors with "numpy.AxisError: axis 6 is out of bounds for array of dimension 0" because `min` was dispatching to `np.min` due to prior `@set_module` decorator.
@blepabyte blepabyte force-pushed the 06-17-fix_transform_order_scope_layers_to_prevent_namespace_pollution branch from fe656f8 to 476ba7b Compare June 17, 2024 08:21
@blepabyte
Copy link
Owner Author

blepabyte commented Jun 17, 2024

Merge activity

  • Jun 17, 4:21 AM EDT: Graphite rebased this pull request after merging its parent, because this pull request is set to merge when ready.
  • Jun 17, 4:23 AM EDT: @blepabyte merged this pull request with Graphite.

@blepabyte blepabyte merged commit 377e1ac into master Jun 17, 2024
2 checks passed
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