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

Update Project.toml #84

Merged
merged 3 commits into from
Jun 26, 2024
Merged

Update Project.toml #84

merged 3 commits into from
Jun 26, 2024

Conversation

jd-lara
Copy link
Member

@jd-lara jd-lara commented Jun 25, 2024

Thanks for opening a PR to PowerNetworkMatrices.jl, please take note of the following when making a PR:

Check the contributor guidelines

Copy link
Contributor

github-actions bot commented Jun 25, 2024

Performance Results

Version Precompile Time
Main 1.082578119
This Branch 1.099065002
Version Execute Time
Main-Build PTDF First 5.40281585
Main-Build PTDF Second 0.110739111
Main-Build Ybus First 0.374718666
Main-Build Ybus Second 0.0078167
Main-Build LODF First 1.187955594
Main-Build LODF Second 0.176774853
This Branch-Build PTDF First 5.330041356
This Branch-Build PTDF Second 0.112402204
This Branch-Build Ybus First 0.380997791
This Branch-Build Ybus Second 0.007204338
This Branch-Build LODF First 0.870689313
This Branch-Build LODF Second 0.364585141

@rodrigomha rodrigomha self-requested a review June 26, 2024 16:49
Copy link
Collaborator

@rodrigomha rodrigomha left a comment

Choose a reason for hiding this comment

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

LGTM

@jd-lara jd-lara merged commit 5e3d6f1 into main Jun 26, 2024
7 checks passed
@jd-lara jd-lara deleted the jd/bump_deps branch October 18, 2024 16:32
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