Skip to content

Use same duration (30 min) for all benchmark signals #90

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.

Sign up for GitHub

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

Jump to bottom
Merged
vegard-solum-4ss merged 3 commits into from
Jun 5, 2025
Merged

Use same duration (30 min) for all benchmark signals #90

vegard-solum-4ss merged 3 commits into from
Jun 5, 2025

Conversation

@vegard-solum-4ss
Copy link
Contributor

@vegard-solum-4ss vegard-solum-4ss commented Jun 5, 2025 •
edited
Loading

Description

Provide a short description about the work that has been done.

Checklist

  • PR title is descriptive and fit for injection into release notes (see tips below).
  • Correct label(s) are used.

PR title tips:

  • Use imperative mood.
  • Describe the motivation for change, issue that has been solved or what has been improved - not how.
  • Examples:
    • Add functionality for Allan variance to smsfusion.simulate
    • Upgrade to support Python 3.10
    • Remove MacOS from CI

@vegard-solum-4ss vegard-solum-4ss changed the title Same duration (30 min) for all benchmark signals Use same duration (30 min) for all benchmark signals Jun 5, 2025
@vegard-solum-4ss vegard-solum-4ss added the enhancement Changes that make the code perform better label Jun 5, 2025
hakon-molven-4ss
hakon-molven-4ss approved these changes Jun 5, 2025
View reviewed changes
Copy link
Contributor

@hakon-molven-4ss hakon-molven-4ss left a comment

Choose a reason for hiding this comment

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

All good

@vegard-solum-4ss vegard-solum-4ss merged commit deb7916 into main Jun 5, 2025
3 checks passed
@vegard-solum-4ss vegard-solum-4ss deleted the benchmark-duration branch June 5, 2025 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hakon-molven-4ss hakon-molven-4ss hakon-molven-4ss approved these changes

Assignees

No one assigned

Labels

enhancement Changes that make the code perform better

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vegard-solum-4ss @hakon-molven-4ss