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

*Expanding integer powers to explicit multiplications #241

Merged
merged 1 commit into from
Nov 18, 2015

Conversation

nikizadehgfdl
Copy link
Contributor

  • This speeds up the ALE routines by 8-10 percent
    But it changes answers
  • These are the Ocean ALE timings for a 1 month OM4_SIS2_COBALT_initFromSrc on eos
before 0.233301    134.104613    112.103448
 after  0.233407    107.906848     89.901184
           min          max           ave

- This speeds up the ALE routines by 8-10 percent
  But it changes answers

- These are the Ocean ALE timings for a 1 month OM4_SIS2_COBALT_initFromSrc on eos
 before 0.233301    134.104613    112.103448
 after  0.233407    107.906848     89.901184
           min          max           ave
adcroft added a commit that referenced this pull request Nov 18, 2015
*Expanding integer powers to explicit multiplications

- Changes answers even for intel. Previous optimizations only changed answers for gnu and pgi.
- Addresses #232
@adcroft adcroft merged commit 328f83c into dev/faster_ale Nov 18, 2015
@nikizadehgfdl nikizadehgfdl deleted the user/nnz/power2multiplication branch July 27, 2016 18:10
gustavo-marques pushed a commit to gustavo-marques/MOM6 that referenced this pull request Apr 12, 2023
* changes in nuopc cap, infra, and MOM.F90 to receive ensembe id from the coupler (alternative to FMS ensemble mngr)

* multi-instance logfile name correction in nuopc cap

* append ensemble suffix to _doc files

* changes in rpointer and restart file name handling to accommodate multi-instance CESM runs

* remove fms2_io_mod usage in FMS1/MOM_ensemble_manager_infra.F90

* rm whitespace in mom_cap
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