Skip to content

Fix a spelling error in the functor name (PotientialVortHAdv->PotentialVortHAdv)#369

Merged
mark-petersen merged 3 commits intoE3SM-Project:developfrom
hyungyukang:hkang/function-name-change
Mar 26, 2026
Merged

Fix a spelling error in the functor name (PotientialVortHAdv->PotentialVortHAdv)#369
mark-petersen merged 3 commits intoE3SM-Project:developfrom
hyungyukang:hkang/function-name-change

Conversation

@hyungyukang
Copy link
Copy Markdown

@hyungyukang hyungyukang commented Mar 19, 2026

This PR fixes a spelling error in the function name, changing PotientialVortHAdv to PotentialVortHAdv in Tendencies.cpp, Tendencies.h, and TimeStepperTest.cpp.

Passed Ctests on Frontier CPU (craycray) and GPU (craycray-mphipcc compilers).

  • Linting
  • Building
    • CMake build does not produce any new warnings from changes in this PR
      Checklist
  • Testing
    • Add a comment to the PR titled Testing with the following:

- Change "PotientialVortHAdv" to "PotentialVortHAdv"
  in Tendencies.cpp, Tendencies.h, and TimeStepperTest.cpp
@hyungyukang hyungyukang changed the title Fix a spelling error in the functor name (PotientialVortHAdv → PotentialVortHAdv) Fix a spelling error in the functor name (PotientialVortHAdv->PotentialVortHAdv) Mar 19, 2026
@alicebarthel
Copy link
Copy Markdown

Yes, thanks! I am so glad to hear it's not just me who was bothered by the typo ;-)

@mark-petersen mark-petersen self-requested a review March 26, 2026 13:30
@mark-petersen mark-petersen self-assigned this Mar 26, 2026
Copy link
Copy Markdown
Collaborator

@mark-petersen mark-petersen left a comment

Choose a reason for hiding this comment

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

Thanks @hyungyukang, this is an obvious one! I merged locally on perlmutter it passed all cTests with gnu on CPUs.

@mark-petersen
Copy link
Copy Markdown
Collaborator

I found three more:

git grep -in Potiential
doc/design/Tendencies.md:48:   PotentialVortHAdvOnEdge PotientialVortHAdv;
src/ocn/Tendencies.cpp:401:      Pacer::start("Tend:potientialVortHAdv", 2);
src/ocn/Tendencies.cpp:415:      Pacer::stop("Tend:potientialVortHAdv", 2);

I'll fix those quick.

@mark-petersen mark-petersen merged commit 29b403d into E3SM-Project:develop Mar 26, 2026
1 check passed
@sbrus89 sbrus89 mentioned this pull request Mar 26, 2026
6 tasks
@mark-petersen
Copy link
Copy Markdown
Collaborator

mark-petersen commented Mar 26, 2026

Note, I altered the ekat and CIME submodules with this merge by mistake. Sorry I missed that. Will fix soon...

@xylar
Copy link
Copy Markdown

xylar commented Mar 27, 2026

@hyungyukang, in the future, it would be helpful if Omega branch names always start with omega. This makes updating Polaris later a lot easier.

xylar added a commit to xylar/polaris that referenced this pull request Mar 27, 2026
This merge updates the e3sm_submodules/Omega submodule from [d0b3482](https://github.com/E3SM-Project/Omega/tree/d0b3482) to [74611d548d](https://github.com/E3SM-Project/Omega/tree/74611d548d).

This update includes the following MPAS-Ocean and MPAS-Frameworks PRs (check mark indicates bit-for-bit with previous PR in the list):
- [ ]  (ocn) E3SM-Project/Omega#343
- [ ]  (ocn) E3SM-Project/Omega#344
- [ ]  (ocn) E3SM-Project/Omega#226
- [ ]  (ocn) E3SM-Project/Omega#369
- [ ]  (ocn) E3SM-Project/Omega#379
xylar added a commit to xylar/polaris that referenced this pull request Mar 27, 2026
This merge updates the e3sm_submodules/Omega submodule from [d0b3482](https://github.com/E3SM-Project/Omega/tree/d0b3482) to [74611d548d](https://github.com/E3SM-Project/Omega/tree/74611d548d).

This update includes the following MPAS-Ocean and MPAS-Frameworks PRs (check mark indicates bit-for-bit with previous PR in the list):
- [ ]  (ocn) E3SM-Project/Omega#343
- [ ]  (ocn) E3SM-Project/Omega#344
- [ ]  (ocn) E3SM-Project/Omega#226
- [ ]  (ocn) E3SM-Project/Omega#369
- [ ]  (ocn) E3SM-Project/Omega#379
@hyungyukang
Copy link
Copy Markdown
Author

@hyungyukang, in the future, it would be helpful if Omega branch names always start with omega. This makes updating Polaris later a lot easier.

Sure, I will. Thanks for letting me know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants