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

Implement airspeed velocity for monitoring esmf-regrid performance #57

Closed
jamesp opened this issue Apr 16, 2021 · 3 comments
Closed

Implement airspeed velocity for monitoring esmf-regrid performance #57

jamesp opened this issue Apr 16, 2021 · 3 comments
Assignees
Labels
New: Feature Highlight a new community raised "feature request" issue

Comments

@jamesp
Copy link
Member

jamesp commented Apr 16, 2021

✨ Feature Request

We should add airspeed velocity performance testing to this package so that we can monitor performance improvements over time.

@jamesp jamesp added the New: Feature Highlight a new community raised "feature request" issue label Apr 16, 2021
@github-actions
Copy link
Contributor

✨ Congratulations! ✨ Thanks for submitting your first issue to iris-esmf-regrid. We really appreciate it and will get back to you as soon as possible. Awesome job 👍

@trexfeathers
Copy link
Contributor

trexfeathers commented May 14, 2021

Agreed steps from conversation with @stephenworsley:

  • Sort out ASV architecture - @trexfeathers working on a PR right now.
  • Write first benchmarks - using sample cube-spheres from SciTools/iris-test-data.
  • Joint and separate experiments with:
  • Apply improved knowledge to decisions on:
    • If/how we integrate some/all benchmarks into continuous integration runs.
      Intelligent use of ASV's repeats should probably provide statistical rigour to compensate for possible risks of running on shared architecture.
    • If we run some/all benchmarks overnight on local hardware.
    • What results we publish, and where.

@trexfeathers
Copy link
Contributor

We now have a minimum architecture in place (follow the breadcrumbs from here), which allows for easy addition of new benchmarks, which form part of the standard CI.

There is the outstanding matter of publishing the results - #90 - but this can come later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New: Feature Highlight a new community raised "feature request" issue
Projects
None yet
Development

No branches or pull requests

2 participants