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

Updates from the package template #481

Merged
merged 2 commits into from
Jan 6, 2025
Merged

Updates from the package template #481

merged 2 commits into from
Jan 6, 2025

Conversation

github-actions[bot]
Copy link
Contributor

This is an autogenerated PR, which will applies the latest changes from the SunPy Package Template.
If this pull request has been opened as a draft there are conflicts which need fixing.

To run the CI on this pull request you will need to close it and reopen it.

@github-actions github-actions bot added the No Changelog Entry Needed This pull request does not need a changelog entry label Dec 30, 2024
@Cadair Cadair closed this Jan 6, 2025
@Cadair Cadair reopened this Jan 6, 2025
Copy link

codspeed-hq bot commented Jan 6, 2025

CodSpeed Performance Report

Merging #481 will improve performances by 23.67%

Comparing cruft/update (6320639) with main (5ad3fda)

Summary

⚡ 9 improvements
✅ 2 untouched benchmarks

Benchmarks breakdown

Benchmark main cruft/update Change
test_dataset_repr 4 ms 3.4 ms +16.43%
test_generate_celestial 5.3 ms 4.4 ms +21.29%
test_load_asdf 1.7 s 1.4 s +22.21%
test_pixel_to_world 174.6 ms 158.5 ms +10.13%
test_plot_dataset[axes0] 5.2 s 4.4 s +18.13%
test_raveled_tab1d_model 1,120.3 µs 958.6 µs +16.87%
test_slice_dataset 65.5 ms 53 ms +23.67%
test_tileddataset_repr 2 ms 1.8 ms +14.6%
test_update_celestial 612.4 µs 525.2 µs +16.62%

.cruft.json Outdated Show resolved Hide resolved
@Cadair Cadair enabled auto-merge (squash) January 6, 2025 10:30
@Cadair Cadair merged commit 66035ba into main Jan 6, 2025
20 of 21 checks passed
@Cadair Cadair deleted the cruft/update branch January 6, 2025 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No Changelog Entry Needed This pull request does not need a changelog entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant