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

Fix Reg Test Speed #1461

Merged
merged 1 commit into from
Jan 27, 2025
Merged

Fix Reg Test Speed #1461

merged 1 commit into from
Jan 27, 2025

Conversation

hgopalan
Copy link
Contributor

@hgopalan hgopalan commented Jan 25, 2025

Changes in input type to speed-up reg test. Tested in debug mode.

old:

nrel_precursor

Time spent in InitData(): 565.0228059
Time spent in Evolve(): 972.1358745

nrel_terrain

Time spent in InitData(): 7.103995917
Time spent in Evolve(): 887.4731385

nrel_terrain_amr

Time spent in InitData(): 7.652189625
Time spent in Evolve(): 1327.320866

new:

nrel_precursor

Time spent in InitData(): 49.61729417
Time spent in Evolve(): 113.3794835

nrel_terrain

Time spent in InitData(): 2.131946458
Time spent in Evolve(): 80.65912675

nrel_terrain_amr

Time spent in InitData(): 2.192530083
Time spent in Evolve(): 151.0545329

Please check the type of change introduced:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • [ X] Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

Checklist

The following is included:

  • new unit-test(s)
  • new regression test(s)
  • documentation for new capability

This PR was tested by running:

  • the unit tests
    • on GPU
    • on CPU
  • the regression tests
    • on GPU
    • [ X] on CPU

Copy link
Contributor

@marchdf marchdf left a comment

Choose a reason for hiding this comment

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

Thanks!

@marchdf marchdf mentioned this pull request Jan 24, 2025
11 tasks
@marchdf marchdf merged commit 9ce018c into Exawind:main Jan 27, 2025
15 checks passed
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