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.

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

Speed test addition #50

Open
lelaus opened this issue Sep 14, 2023 · 2 comments
Open

Speed test addition #50

lelaus opened this issue Sep 14, 2023 · 2 comments
Assignees
@panDanieleN
Labels
enhancement New feature or request

Comments

@lelaus
Copy link
Collaborator

lelaus commented Sep 14, 2023 •
edited

  • [1] add a speed test to the test suite for performance tracking purposes
@panDanieleN panDanieleN added the enhancement New feature or request label Sep 14, 2023
@lelaus lelaus assigned panDanieleN and unassigned lelaus Oct 3, 2023
@lelaus
Copy link
Collaborator Author

lelaus commented Oct 3, 2023 •
edited

In detail:

  1. to set the target functions to test;
  2. to include the speed test among the regression tests made when pushing new updates to master branch;
  3. to take memory about the speed test results across successive pushes;
  4. saving artifacts: txt formatted with test name on the rows and performance times on the columns, each one for each column
  5. artifacts: it should be only local within the repository (the performance comparison makes sense only within the same platform)

@panDanieleN
Copy link
Collaborator

From main action #202 the execution speed of the tests slowed down, probably due to the type check method decorator. Need to decide whether to keep the check or not

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@panDanieleN panDanieleN

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@panDanieleN @lelaus