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

chore: Change GEOSX to GEOS in a bunch of places #3219

Merged
merged 24 commits into from
Jul 19, 2024

Conversation

rrsettgast
Copy link
Member

@rrsettgast rrsettgast commented Jul 10, 2024

Changes were incrementally committed:

  • GEOSX_USE to GEOS_USE
  • GEOSX_ENABLE to GEOS_ENABLE
  • GEOSX_LA to GEOS_LA
  • GEOSX_TPL to GEOS_TPL
  • GEOSX_ to GEOS_
  • Update copyright statements

Sorry y'all....this PR is quite unreviewable. There should be no functional code changes.

@rrsettgast rrsettgast force-pushed the cleanup/Macros-GEOSX_XXX-to-GEOS_XXX branch from cc35dc1 to 5065190 Compare July 10, 2024 22:24
@rrsettgast rrsettgast closed this Jul 10, 2024
@rrsettgast rrsettgast force-pushed the cleanup/Macros-GEOSX_XXX-to-GEOS_XXX branch from c8174b9 to 6c82021 Compare July 10, 2024 22:37
@rrsettgast rrsettgast reopened this Jul 10, 2024
@rrsettgast rrsettgast self-assigned this Jul 11, 2024
@rrsettgast rrsettgast added ci: run CUDA builds Allows to triggers (costly) CUDA jobs ci: run integrated tests Allows to run the integrated tests in GEOS CI and removed ci: run integrated tests Allows to run the integrated tests in GEOS CI labels Jul 11, 2024
Copy link
Contributor

@MelReyCG MelReyCG left a comment

Choose a reason for hiding this comment

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

What about the file names? Is it for a future PR?

rrsettgast and others added 7 commits July 18, 2024 13:21
* Update Publications.rst

* Update Publications.rst

---------

Co-authored-by: Ryan Aronson <raronson0031@gmail.com>
Co-authored-by: Nicola Castelletto <38361926+castelletto1@users.noreply.github.com>
Co-authored-by: Randolph Settgast <settgast1@llnl.gov>
* fix: do Not require successful conventional semantic job

---------

Co-authored-by: Lionel Untereiner <lionel.untereiner@geosiris.com>
* update blt to 0.6.2

* update RAJA and CHAI to v2024.02.2

* update Caliper to 2.11.0

*update adiak to 0.4.0

* update conduit to 0.9.2

* change to new HDF import. use umpires fmt and add extra dependency to executable

* changes for change in raja interface

* move src/coreComponents/unitTests/dataRepository/testPacking.cpp to src/coreComponents/dataRepository/unitTests

* switched to syncronous packing/upacking to avoid test failures on CI image running on streak
@rrsettgast rrsettgast force-pushed the cleanup/Macros-GEOSX_XXX-to-GEOS_XXX branch from 97feaaf to 15a46f3 Compare July 18, 2024 20:49
@rrsettgast rrsettgast added the ci: run integrated tests Allows to run the integrated tests in GEOS CI label Jul 19, 2024
@rrsettgast rrsettgast merged commit 18db014 into develop Jul 19, 2024
41 of 43 checks passed
@rrsettgast rrsettgast deleted the cleanup/Macros-GEOSX_XXX-to-GEOS_XXX branch July 19, 2024 16:06
Algiane pushed a commit that referenced this pull request Jul 30, 2024
* GEOSX_USE to GEOS_USE

* GEOSX_ENABLE to GEOS_ENABLE

* GEOSX_LA to GEOS_LA

* GEOSX_TPL to GEOS_TPL

* GEOSX_ to GEOS_ excluding .py and .msub

* change version number
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci: run CUDA builds Allows to triggers (costly) CUDA jobs ci: run integrated tests Allows to run the integrated tests in GEOS CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants