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

added arcsinh to numpy #3805

Merged
merged 11 commits into from
Aug 30, 2022
Merged

added arcsinh to numpy #3805

merged 11 commits into from
Aug 30, 2022

Conversation

jvt3112
Copy link
Contributor

@jvt3112 jvt3112 commented Aug 30, 2022

Copy link
Contributor

@Daniel4078 Daniel4078 left a comment

Choose a reason for hiding this comment

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

Hello and I have found several things that you need to change before this pull request can be approved. Please check the comments below for more details. Thank you.

Copy link
Contributor

@Daniel4078 Daniel4078 left a comment

Choose a reason for hiding this comment

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

Firstly, I do not think you should include the init.pyc file in this pull request.
Secondly, you have some formatting errors with your code:
ivy_tests/test_ivy/test_frontends/test_numpy/test_mathematical_functions/test_np_hyperbolic_functions.py:113:1: E302 expected 2 blank lines, found 1
ivy_tests/test_ivy/test_frontends/test_numpy/test_mathematical_functions/test_np_hyperbolic_functions.py:117:5: E128 continuation line under-indented for visual indent
ivy_tests/test_ivy/test_frontends/test_numpy/test_mathematical_functions/test_np_hyperbolic_functions.py:125:13: E123 closing bracket does not match indentation of opening bracket's line
ivy_tests/test_ivy/test_frontends/test_numpy/test_mathematical_functions/test_np_hyperbolic_functions.py:126:9: E123 closing bracket does not match indentation of opening bracket's line
ivy_tests/test_ivy/test_frontends/test_numpy/test_mathematical_functions/test_np_hyperbolic_functions.py:130:1: E302 expected 2 blank lines, found 1
ivy_tests/test_ivy/test_frontends/test_numpy/test_mathematical_functions/test_np_hyperbolic_functions.py:132:17: E251 unexpected spaces around keyword / parameter equals
ivy_tests/test_ivy/test_frontends/test_numpy/test_mathematical_functions/test_np_hyperbolic_functions.py:159:13: E251 unexpected spaces around keyword / parameter equals
ivy_tests/test_ivy/test_frontends/test_numpy/test_mathematical_functions/test_np_hyperbolic_functions.py:159:15: E251 unexpected spaces around keyword / parameter equals
Secondly, TypeError: arcsinh() got an unexpected keyword argument 'axis' in your test, so you should not have axis passed into the test. This also implies that you should not just copy and paste from other examples but you need to change the code according the arcsinh function, which does not have an axis and need where argument the same size as the input array. Thank you.

Copy link
Contributor

@Daniel4078 Daniel4078 left a comment

Choose a reason for hiding this comment

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

well done, I think there is nothing wrong with your code now and the pull request can be merged.

@Daniel4078 Daniel4078 merged commit 933731b into ivy-llc:master Aug 30, 2022
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