Skip to content

Fix erroneous gaussian quadrature points in gauss_legendre - #660

Merged
milancurcic merged 2 commits into
fortran-lang:masterfrom
hsnyder:master
Jun 13, 2022
Merged

milancurcic merged 2 commits into
fortran-lang:masterfrom
hsnyder:master

Conversation

@hsnyder

@hsnyder hsnyder commented May 23, 2022

Copy link
Copy Markdown
Contributor

fixes #619

@jvdp1 jvdp1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Thank you.

@jvdp1
jvdp1 requested review from a team and ivan-pi May 23, 2022 12:07

@milancurcic milancurcic left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, thank you, I left one suggestion for change.

Comment thread src/tests/quadrature/test_gauss.f90
@milancurcic

Copy link
Copy Markdown
Member

Thanks! Will merge.

@milancurcic
milancurcic merged commit d110944 into fortran-lang:master Jun 13, 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.

gauss_legendre with interval

3 participants