Skip to content

Added the axis variable to new functions#1

Open
lrnv wants to merge 1 commit intogbb:masterfrom
lrnv:adding_axis
Open

Added the axis variable to new functions#1
lrnv wants to merge 1 commit intogbb:masterfrom
lrnv:adding_axis

Conversation

@lrnv
Copy link

@lrnv lrnv commented Jan 10, 2020

I used new functions names to not have to handle the case axis = None and not slow up computations when the axis parameter is not needed.

I did not run tests, but this should run fine.

Thanks for the nice code by the way, this is simple but stunning.

I used new functions to not have to handle the case `axis = None` and not slow up computations.
@gbb
Copy link
Owner

gbb commented Jan 12, 2020 via email

@lrnv
Copy link
Author

lrnv commented Jan 28, 2020

Hey, did not hear from you ;

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