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

create new test case for ccd #11

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

create new test case for ccd #11

wants to merge 11 commits into from

Conversation

zywshoon
Copy link
Collaborator

No description provided.

@zywshoon zywshoon requested a review from fvitt September 17, 2024 22:34
@zywshoon
Copy link
Collaborator Author

I have no idea what to do with these 1000 changes... I only changed Makefile and add a new test case in tests/.

@fvitt
Copy link
Collaborator

fvitt commented Sep 18, 2024

@zywshoon
It seems that you added source/.DS_Store, build, doc/f90doc, and run in your commit to your branch. You should not add these. Try removing them from your branch via git rm source/.DS_Store, git rm build, etc. Then commit and push to your github fork.

@zywshoon
Copy link
Collaborator Author

Ok. It is reasonable now. Let me know what's next. I don't think all the changes should be adopted since some setup in Makefile is only for running on my own computer.

@fvitt
Copy link
Collaborator

fvitt commented Sep 18, 2024

@zywshoon

I will clone your branch on casper when it is back up. There I will create the new baselines.

Something happened to the doc/ChangeLog, doc/ChangeLog_template, and doc/index.html files in your branch. I will restore those.

I will also remove the changes to the Makefile you did for your computer.

Then I will try to push to your github fork / branch.

@fvitt
Copy link
Collaborator

fvitt commented Sep 24, 2024

@zywshoon
I am seeing differences in these bench data files:

        modified:   tests/bench/carma_fractalmicrotest.txt
        modified:   tests/bench/carma_sulfate_vehkamaki_test.txt
        modified:   tests/bench/carma_sulfatetest.txt
        modified:   tests/bench/carma_sulfhet_vehkamaki_test.txt
        modified:   tests/bench/carma_sulfhettest.txt

And there is 1 new file:

        new file:   tests/bench/carma_sigmavdry.txt

Is this what you expect?

@zywshoon
Copy link
Collaborator Author

@zywshoon I am seeing differences in these bench data files:

        modified:   tests/bench/carma_fractalmicrotest.txt
        modified:   tests/bench/carma_sulfate_vehkamaki_test.txt
        modified:   tests/bench/carma_sulfatetest.txt
        modified:   tests/bench/carma_sulfhet_vehkamaki_test.txt
        modified:   tests/bench/carma_sulfhettest.txt

And there is 1 new file:

        new file:   tests/bench/carma_sigmavdry.txt

Is this what you expect?

The modified bench .txt files look correct to me.

carma_sigmavdry.txt is not the new file. the new file should be carma_sulfate_ccdon.F90 and carma_sulfate_ccdon.txt

Did I miss uploading them?

@fvitt
Copy link
Collaborator

fvitt commented Sep 24, 2024

@zywshoon
My mistake. It turns out that the name of the new test executable needs to end with TEST.EXE for run-regress.csh to work correctly. So I renamed your new test to include _test.
Now I have :

        new file:   tests/bench/carma_sulfate_ccdon_test.txt
        renamed:    tests/carma_sulfate_ccdon.F90 -> tests/carma_sulfate_ccdon_test.F90

        modified:   tests/bench/carma_fractalmicrotest.txt
        modified:   tests/bench/carma_sulfate_vehkamaki_test.txt
        modified:   tests/bench/carma_sulfatetest.txt
        modified:   tests/bench/carma_sulfhet_vehkamaki_test.txt
        modified:   tests/bench/carma_sulfhettest.txt
        modified:   tests/carma_sulfate_ccdon_test.F90

Do these change make sense now?

@zywshoon
Copy link
Collaborator Author

@zywshoon My mistake. It turns out that the name of the new test executable needs to end with TEST.EXE for run-regress.csh to work correctly. So I renamed your new test to include _test. Now I have :

        new file:   tests/bench/carma_sulfate_ccdon_test.txt
        renamed:    tests/carma_sulfate_ccdon.F90 -> tests/carma_sulfate_ccdon_test.F90

        modified:   tests/bench/carma_fractalmicrotest.txt
        modified:   tests/bench/carma_sulfate_vehkamaki_test.txt
        modified:   tests/bench/carma_sulfatetest.txt
        modified:   tests/bench/carma_sulfhet_vehkamaki_test.txt
        modified:   tests/bench/carma_sulfhettest.txt
        modified:   tests/carma_sulfate_ccdon_test.F90

Do these change make sense now?

Yes. These look correct to me.

…revious tags

        new file:   tests/bench/carma_sulfate_ccdon_test.txt
        renamed:    tests/carma_sulfate_ccdon.F90 -> tests/carma_sulfate_ccdon_test.F90

        modified:   Makefile
        modified:   doc/ChangeLog
        modified:   tests/Makefile
        modified:   tests/bench/carma_fractalmicrotest.txt
        modified:   tests/bench/carma_sulfate_vehkamaki_test.txt
        modified:   tests/bench/carma_sulfatetest.txt
        modified:   tests/bench/carma_sulfhet_vehkamaki_test.txt
        modified:   tests/bench/carma_sulfhettest.txt
        modified:   tests/carma_sulfate_ccdon_test.F90
        modified:   doc/ChangeLog
        modified:   source/base (new commits)
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.

2 participants