The start point for the C3P Community Cloud-Convection Parameterization and Merging with MONAN v1.4.1-rc #16
Merged
joaomas merged 46 commits intomonanadmin:developfrom Aug 12, 2025
Merged
Conversation
Contributor
|
Assigned #791 for testing.
|
Contributor
|
@saulorfreitas the test with the debug flag turned on an error, the message is: "model.e.XXX"
|
Contributor
|
@saulorfreitas the test folder is: "/mnt/beegfs/joao.messias/pr_teste_debug/scripts_CD-CT/dataout/" Thanks. |
Contributor
Author
|
@joaomas bug fixed. Please, check my develop. Do I need to make a new pull request? |
Contributor
Author
|
Ola João,
Corrigi o bug. Era divisão zero/zero. Já atualizei a minha develop.
Obrigado e desculpe-me por este bug.
Saulo
|
Contributor
|
Testing again... |
joaomas
requested changes
Aug 8, 2025
Contributor
joaomas
left a comment
There was a problem hiding this comment.
The files in the folder "/own_namelist_monan" and "src/core_atmosphere/physics/physics_monan/own_files/module_cu_gf.monan.F_1.3" e "src/core_atmosphere/physics/physics_monan/own_files/module_cu_gf.monan.F_1.4" need to be removed by you.
instructions on this sent by email.
src/core_atmosphere/physics/physics_monan/own_files/module_cu_gf.monan.F_1.3
Outdated
Show resolved
Hide resolved
src/core_atmosphere/physics/physics_monan/own_files/module_cu_gf.monan.F_1.4
Outdated
Show resolved
Hide resolved
joaomas
approved these changes
Aug 12, 2025
Contributor
joaomas
left a comment
There was a problem hiding this comment.
- Test Compile: OK
- Tests Execution: OK
2.1. Test 1: 20250716 12Z - 240h FCST - 512c8n: OK
2.2. Test 2: 20250717 12Z - 240h FCST - 256c4n: OK
2.3. Test 3: 20250716 12Z - flag "debug=true" - 6h FCST - 512c8n: OK
2.4. Test 4: 20250717 12Z - flag "debug=true" - 6h FCST - 2562c4n: OK
2.5. Test 5: 20240420 00Z - first day P1 - 240h FCST - 512c8n: OK - Compare with last version: OK
- DTN-01 Check: not applied yet
Pull Request approved.
This file contains hidden or 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
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.


**** Attention: disregard all files inside the “own_...” directories **** . Those files are only for my own work.
This new version of GF contains: