Develop #2
Merged
Develop #2
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Decided to rename ALL functions and subroutines to be prefixed with ccpp_. This also involved renaming all source files to be prefixed as well. Updated the fields index routines so as to add a field without having to specify the rank and shape. Created a test directory within the src directory to hold all the test cases. Rejigged the schema XML parser to be able to handle the different/optional fields and attributes of the schema.
Updating the Travis yaml file to include building and testing the develop branch.
Adding a ccpp toplevel driver/init/fini routine for the library. Removing the C types header as it is not needed anymore. Expanded the CCPP fields subroutines to include integer 32, 64 and real 32, 64 bits. Using iso_fortran_env, so as to be able to remove the kinds module in the future. Expanded the use statements in the ccpp_ipd module to one pull in what it needs. Renamed the ccpp_phy_infra module to be ccpp_suite as that is what it really focuses on.
Updated the README.md to include instrutions for the re-arranged code base. Removed the dependencies on kinds.f90. Completed the ccpp_suite_fini() subroutine.
Removing all dependencies on the kinds module.
Cleaned-up the CMakeLists.txt, so the project installs properly. Bug fix in ccpp_suite with assumed lenght strings. Added a logical data field to the ccpp_fields Cleaned-up the test case, all arrays (in the atm cap) have to have the target attribute and all arrays (in the physics cap) must have the pointer attribute.
I had missed off half a dozen "target" attributes for the subroutines to add a field when using c_loc (thanks GCC 6.3!). Also change the "module procedure" definition for the add and get interfaces so as to compile with PGI.
Cleaned up the Doxygen documentation, however derivered types are not getting documented.
Theia has has cmake version 2.8.12.2 which does not support Policy cmp0048. Adding the project info and version into an if block based on cmp0048.
Adding CPack to the mix, so as to be able to create packages. Setting the build of a shared library by default.
bluefinweiwei
pushed a commit
to bluefinweiwei/ccpp-framework
that referenced
this pull request
May 23, 2019
Features/ccpp doc chap4
climbfuji
added a commit
that referenced
this pull request
Oct 31, 2019
Bugfix #2 for travis test in schemes/check/ccpp_prebuild_config.py
climbfuji
referenced
this pull request
in climbfuji/ccpp-framework
Nov 4, 2019
Bugfix #2 for travis test in schemes/check/ccpp_prebuild_config.py
climbfuji
referenced
this pull request
in climbfuji/ccpp-framework
Feb 13, 2020
gsd/develop: update from master 2020/01/27
nusbaume
pushed a commit
to nusbaume/ccpp-framework
that referenced
this pull request
Jul 14, 2023
Unit test actions Adds capgen unit tests to the list of actions.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Committing the changes to overhaul CCPP into a library that has the ccpp_ prefix and is easier to use.