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

Adds notes about generating surface dataset and initial condition for ELM #6582

Merged
merged 7 commits into from
Oct 15, 2024

Conversation

bishtgautam
Copy link
Contributor

@bishtgautam bishtgautam commented Sep 3, 2024

Adds notes on using the following ELM tools:

  • mksurfdata_map: For creating a surface dataset
  • interpinic: For creating an initial condition.

[BFB]

Copy link

github-actions bot commented Sep 3, 2024

PR Preview Action v1.4.8
🚀 Deployed preview to https://E3SM-Project.github.io/E3SM/pr-preview/pr-6582/
on branch gh-pages at 2024-10-09 11:31 UTC

@bishtgautam bishtgautam force-pushed the bishtgautam/lnd/interpinic-notes branch from 31edacd to 5b0a222 Compare September 3, 2024 21:51
@bishtgautam bishtgautam force-pushed the bishtgautam/lnd/interpinic-notes branch 2 times, most recently from ab88c97 to 76ebbcd Compare September 18, 2024 15:18
@bishtgautam bishtgautam force-pushed the bishtgautam/lnd/interpinic-notes branch from 76ebbcd to f61c6f8 Compare September 18, 2024 15:19
@bishtgautam bishtgautam changed the title [WIP] Adds notes about generating ELM land IC Adds notes about generating surface dataset and initial condition for ELM Sep 18, 2024
eval $(./cime/CIME/Tools/get_case_env/get_case_env)

# Change dir to `interpinic`
cd components/elm/tools/interpinic/src
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When I tried this, the executable was one directory up (not in /src).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed.


# Load relevant modules.
cd <e3sm-dir>
eval $(./cime/CIME/Tools/get_case_env/get_case_env)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

extra 'get_case_env' in path.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed.


USER_FC=ifort LIB_NETCDF="`nc-config --flibs`" INC_NETCDF="`nf-config --includedir`" make VERBOSE=1
```

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cd ..

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed.


export USER_LDFLAGS="-L$NETCDF_C_DIR/lib -lnetcdf -L$NETCDF_F_DIR/lib -lnetcdff -L$HDF5_DIR/lib -lhdf5"

USER_FC=ifort LIB_NETCDF="`nc-config --flibs`" INC_NETCDF="`nf-config --includedir`" make VERBOSE=1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing "export" ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

export is not required.

@rljacob
Copy link
Member

rljacob commented Oct 4, 2024

@bishtgautam see comments on this PR.

@rljacob
Copy link
Member

rljacob commented Oct 9, 2024

@bishtgautam you can merge this.

bishtgautam added a commit that referenced this pull request Oct 14, 2024
Adds notes on using the following ELM tools:

- `mksurfdata_map`: For creating a surface dataset
- `interpinic`: For creating an initial condition.

[BFB]
@bishtgautam bishtgautam merged commit eb9efce into master Oct 15, 2024
10 checks passed
@bishtgautam bishtgautam deleted the bishtgautam/lnd/interpinic-notes branch October 15, 2024 22:22
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.

5 participants