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

Add performance checks #24

Merged
merged 5 commits into from
Jun 4, 2024
Merged

Add performance checks #24

merged 5 commits into from
Jun 4, 2024

Conversation

Azzaare
Copy link
Member

@Azzaare Azzaare commented Jun 1, 2024 •
edited
Loading

This PR propose basic checks using PerfChecker.

@VarLad , we might have some tweaks to do on PerfChecker.jl to make the output in this PR better:

  • Propose (default?) log scale
  • Handle the cases where there is no allocations in the package being checked
  • Include the output of visuals in the dictionarry option
  • Add a "make_visual" function and or macro to produce visuals based on existing output

@Azzaare Azzaare added bug Something isn't working enhancement New feature or request help wanted Extra attention is needed labels Jun 1, 2024
@Azzaare Azzaare changed the base branch from main to dev June 1, 2024 05:12
@Azzaare Azzaare merged commit bf5d52a into dev Jun 4, 2024
11 checks passed
@Azzaare Azzaare deleted the perfchecks branch June 4, 2024 05:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Azzaare Azzaare

@VarLad VarLad

Labels
bug Something isn't working enhancement New feature or request help wanted Extra attention is needed
Projects
Opt Pot
Status: Done
PerfChecker.jl
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Azzaare @VarLad