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

I have extracted Arange method from GridPointUtility. Please add … #179

Merged
merged 1 commit into from
Dec 12, 2022

Conversation

vickysharma0812
Copy link
Member

@vickysharma0812 vickysharma0812 commented Dec 12, 2022

I have extracted Arange method from GridPointUtility. Please add … more methods to Arange, See functionalFortran.

Next:

  • We will extracts LinSpace, LogSpace also
  • This provides a very good structure to Utility module.
  • Also, in this way, it is easy to include the functionalFortran library.

Changes to be committed:
modified: src/modules/Utility/CMakeLists.txt
new file: src/modules/Utility/src/ArangeUtility.F90
modified: src/modules/Utility/src/GridPointUtility.F90
modified: src/modules/Utility/src/Utility.F90
modified: src/submodules/Utility/CMakeLists.txt
new file: src/submodules/Utility/src/ArangeUtility@Methods.F90
modified: src/submodules/Utility/src/GridPointUtility@Methods.F90

…more methods to Arange, See functionalFortran.

Next:

- We will extracts `LinSpace`, `LogSpace` also
- This provides a very good structure to `Utility` module.
- Also, in this way, it is easy to include the functionalFortran library.

Changes to be committed:
modified:   src/modules/Utility/CMakeLists.txt
new file:   src/modules/Utility/src/ArangeUtility.F90
modified:   src/modules/Utility/src/GridPointUtility.F90
modified:   src/modules/Utility/src/Utility.F90
modified:   src/submodules/Utility/CMakeLists.txt
new file:   src/submodules/Utility/src/ArangeUtility@Methods.F90
modified:   src/submodules/Utility/src/GridPointUtility@Methods.F90
@vickysharma0812 vickysharma0812 merged commit 3523350 into master Dec 12, 2022
vickysharma0812 added a commit that referenced this pull request Dec 16, 2023
- Fixes issue #179
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant