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

Update symbolic link from GCClassic root folder to the test folder #41

Merged
merged 2 commits into from
Jul 5, 2023

Conversation

yantosca
Copy link
Contributor

@yantosca yantosca commented Jun 28, 2023

Name and Institution (Required)

Name: Bob Yantosca
Institution: Harvard + GCST

Confirm you have reviewed the following documentation

Describe the update

This is a companion PR to geoschem/geos-chem#1794. This resets the test folder to the new path src/GEOS-Chem/test to be consistent with the changes made in the geoschem/geos-chem repo.

Expected changes

This is a zero-diff update, as it only modifies a symbolic link. No scientific changes are added.

Reference(s)

N/A

Related Github Issue(s)

Please link to the corresponding Github issue here. If fixing a bug, there should be an issue describing it with steps to reproduce.

Change symbolic link:
 src/GEOS-Chem/test/GCClassic

to
 src/GEOS-Chemtest

since this new test folder now contains the following directories:

 difference/
 integration/
 parallel/
 shared/

To get to the GEOS-Chem integration test script folder, use:

 cd GCClassic/test/integration/GCClassic

To get to the GEOS-Chem parallel test script folder, use:

 cd GCClassic/test/parallel/GCClassic

Signed-off-by: Bob Yantosca <[email protected]>
@yantosca yantosca added the no-diff-to-benchmark This update will not change the results of fullchem benchmark simulations label Jun 28, 2023
@yantosca yantosca added this to the 14.2.1 milestone Jun 28, 2023
@yantosca yantosca requested a review from msulprizio June 28, 2023 20:24
@yantosca yantosca self-assigned this Jun 28, 2023
CHANGELOG.md
- Added sentence about the 'test' folder being repointed to
  'src/GEOS-Chem/test'.

Signed-off-by: Bob Yantosca <[email protected]>
@yantosca yantosca added the topic: Structural Modifications Structural modifications (as opposed to scientific updates) label Jun 28, 2023
@yantosca yantosca merged commit 8b373e5 into dev/14.2.1 Jul 5, 2023
@yantosca yantosca deleted the feature/int-test-rundir-updates branch July 5, 2023 17:19
@yantosca
Copy link
Contributor Author

yantosca commented Jul 5, 2023

Merged upon successful integration tests (and confirmation of zero-diff results) of geoschem/geos-chem#1794.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-diff-to-benchmark This update will not change the results of fullchem benchmark simulations topic: Structural Modifications Structural modifications (as opposed to scientific updates)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant