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

test: strengthen tests for bessely0 function #3712

Merged
merged 4 commits into from
Mar 27, 2024

Conversation

parth121101
Copy link
Contributor

As suggested by @Pranavchiku I have added more tests for bessel_y0 function for rigorous testing.

Copy link
Contributor

@Pranavchiku Pranavchiku left a comment

Choose a reason for hiding this comment

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

Requested changes above.

@Pranavchiku Pranavchiku changed the title Added more tests for bessel_y0 function test: strengthen tests for bessely0 function Mar 26, 2024
@Pranavchiku Pranavchiku added tests intrinsic Issue or pull request specific to intrinsic function labels Mar 26, 2024
@Pranavchiku Pranavchiku changed the title test: strengthen tests for bessely0 function test: strengthen tests for bessely0 function Mar 26, 2024
@parth121101
Copy link
Contributor Author

Requested changes above.

Done @Pranavchiku, thanks for letting me know a better way to manage this tests

@Pranavchiku
Copy link
Contributor

Would you mind adding test for large positive values?

@parth121101
Copy link
Contributor Author

parth121101 commented Mar 26, 2024

Would you mind adding test for large positive values?

I will add them @Pranavchiku

@Pranavchiku Pranavchiku added the needs rebase Rebase on top of latest main is required/desired label Mar 27, 2024
@parth121101 parth121101 marked this pull request as ready for review March 27, 2024 12:02
@Pranavchiku Pranavchiku removed the needs rebase Rebase on top of latest main is required/desired label Mar 27, 2024
Comment on lines +21 to +23
expected = [ 7.9836428649154907E-155_dp,-7.0751499885966513E-155_dp, 4.4632457212022053E-155_dp, -7.7717941781117305E-156_dp, &
-2.1476418943648812E-002_dp, 0.45752410314844527_dp, -3.6134118144921369E-003_dp, -1.1810118244863275_dp, &
5.7258225070763252E-002_dp, -1.9980991452871046E-002_dp, 0.27136794388300378_dp, 0.26703177612919299_dp, &
Copy link
Contributor

Choose a reason for hiding this comment

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

same comment here

Copy link
Contributor

@Pranavchiku Pranavchiku left a comment

Choose a reason for hiding this comment

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

Minor comments, do that and this is good to merge.

@certik certik merged commit 03c57e0 into lfortran:main Mar 27, 2024
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
intrinsic Issue or pull request specific to intrinsic function tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants