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

Post-release 2024.11.20 #2887

Merged
merged 2 commits into from
Nov 20, 2024
Merged

Conversation

tardis-bot
Copy link
Contributor

*beep* *bop*

Hi, human.

These are the changes required after the latest TARDIS release.

⚠️ Warning:

The post_release workflow waits for a webhook and then updates the citation files. If the file is not modified by this pull request, please check the latest release on Zenodo's website, close this pull request, and manually run the workflow.

This pull request should be auto-merged.

@epassaro epassaro enabled auto-merge (squash) November 20, 2024 16:09
@tardis-bot
Copy link
Contributor Author

tardis-bot commented Nov 20, 2024

*beep* *bop*
Hi human,
I ran ruff on the latest commit (8a5e782).
Here are the outputs produced.
Results can also be downloaded as artifacts here.
Summarised output:

3	E999	SyntaxError: Expected a statement

Complete output(might be large):

CITATION.cff:3:1: E999 SyntaxError: Invalid annotated assignment target
README.rst:1:1: E999 SyntaxError: Expected a statement
docs/resources/credits.rst:1:1: E999 SyntaxError: Expected a statement
Found 3 errors.

Copy link

codecov bot commented Nov 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.87%. Comparing base (20b8120) to head (8a5e782).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2887      +/-   ##
==========================================
- Coverage   70.24%   69.87%   -0.38%     
==========================================
  Files         214      214              
  Lines       15981    15981              
==========================================
- Hits        11226    11166      -60     
- Misses       4755     4815      +60     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

@tardis-bot
Copy link
Contributor Author

tardis-bot commented Nov 20, 2024

*beep* *bop*
Hi human,
I ran benchmarks as you asked comparing master (20b8120) and the latest commit (8a5e782).
Here are the logs produced by ASV.
Results can also be downloaded as artifacts here.

Significantly changed benchmarks:

All benchmarks:

Benchmarks that have stayed the same:

| Change   | Before [20b8120b] <release-2024.11.20>   | After [8a5e7823]    | Ratio   | Benchmark (Parameter)                                                                                                               |
|----------|------------------------------------------|---------------------|---------|-------------------------------------------------------------------------------------------------------------------------------------|
|          | 41.3±20μs                                | 46.4±20μs           | ~1.12   | transport_montecarlo_interaction.BenchmarkTransportMontecarloInteraction.time_line_emission                                         |
|          | 6.91±0.8μs                               | 6.09±0.8μs          | ~0.88   | transport_montecarlo_vpacket.BenchmarkMontecarloMontecarloNumbaVpacket.time_trace_vpacket                                           |
|          | 30.7±6μs                                 | 24.7±7μs            | ~0.80   | transport_montecarlo_packet_trackers.BenchmarkTransportMontecarloPacketTrackers.time_generate_rpacket_last_interaction_tracker_list |
|          | 9.30±1μs                                 | 6.76±2μs            | ~0.73   | transport_montecarlo_vpacket.BenchmarkMontecarloMontecarloNumbaVpacket.time_trace_vpacket_volley                                    |
|          | 4.14±2μs                                 | 2.95±0.5μs          | ~0.71   | transport_montecarlo_vpacket.BenchmarkMontecarloMontecarloNumbaVpacket.time_trace_vpacket_within_shell                              |
|          | 2.96±0.6μs                               | 1.35±0.5μs          | ~0.46   | transport_geometry_calculate_distances.BenchmarkTransportGeometryCalculateDistances.time_calculate_distance_line                    |
|          | 2.04±1μs                                 | 2.15±1μs            | 1.05    | transport_montecarlo_estimators_radfield_estimator_calcs.BenchmarkMontecarloMontecarloNumbaPacket.time_update_line_estimators       |
|          | 3.65±0.01ms                              | 3.71±0ms            | 1.02    | opacities_opacity_state.BenchmarkOpacitiesOpacityState.time_opacity_state_initialize('macroatom')                                   |
|          | 64.7±0.08ms                              | 66.1±0.1ms          | 1.02    | transport_montecarlo_packet_trackers.BenchmarkTransportMontecarloPacketTrackers.time_rpacket_trackers_to_dataframe                  |
|          | 30.7±0.01μs                              | 31.2±0.03μs         | 1.01    | transport_montecarlo_packet_trackers.BenchmarkTransportMontecarloPacketTrackers.time_generate_rpacket_tracker_list                  |
|          | 3.26±0.9μs                               | 3.28±0.6μs          | 1.01    | transport_montecarlo_vpacket.BenchmarkMontecarloMontecarloNumbaVpacket.time_trace_bad_vpacket                                       |
|          | 2.07±0m                                  | 2.07±0m             | 1.00    | spectrum_formal_integral.BenchmarkTransportMontecarloFormalIntegral.time_FormalIntegrator_functions                                 |
|          | 205±0.1ns                                | 205±0.06ns          | 1.00    | spectrum_formal_integral.BenchmarkTransportMontecarloFormalIntegral.time_intensity_black_body                                       |
|          | 1.23±0μs                                 | 1.22±0μs            | 0.99    | transport_geometry_calculate_distances.BenchmarkTransportGeometryCalculateDistances.time_calculate_distance_boundary                |
|          | 745±0.7ns                                | 737±0.4ns           | 0.99    | transport_montecarlo_interaction.BenchmarkTransportMontecarloInteraction.time_thomson_scatter                                       |
|          | 572±100ns                                | 561±100ns           | 0.98    | opacities_opacity.BenchmarkMontecarloMontecarloNumbaOpacities.time_pair_creation_opacity_calculation                                |
|          | 1.06±0m                                  | 1.05±0m             | 0.98    | run_tardis.BenchmarkRunTardis.time_run_tardis_rpacket_tracking                                                                      |
|          | 43.0±30μs                                | 42.2±20μs           | 0.98    | transport_montecarlo_interaction.BenchmarkTransportMontecarloInteraction.time_line_scatter                                          |
|          | 40.0±0s                                  | 39.0±0.01s          | 0.97    | run_tardis.BenchmarkRunTardis.time_run_tardis                                                                                       |
|          | 1.77±0.01ms                              | 1.71±0.01ms         | 0.96    | transport_montecarlo_main_loop.BenchmarkTransportMontecarloMontecarloMainLoop.time_montecarlo_main_loop                             |
|          | 621±100ns                                | 590±200ns           | 0.95    | opacities_opacity.BenchmarkMontecarloMontecarloNumbaOpacities.time_photoabsorption_opacity_calculation                              |
|          | 2.71±0.4ms                               | 2.57±0.4ms          | 0.95    | transport_montecarlo_single_packet_loop.BenchmarkTransportMontecarloSinglePacketLoop.time_single_packet_loop                        |
|          | 541±100ns                                | 511±100ns           | 0.94    | opacities_opacity.BenchmarkMontecarloMontecarloNumbaOpacities.time_compton_opacity_calculation                                      |
|          | 2.77±0.01ms                              | 2.56±0.02ms         | 0.92    | opacities_opacity_state.BenchmarkOpacitiesOpacityState.time_opacity_state_initialize('scatter')                                     |

If you want to see the graph of the results, you can check it here

@epassaro epassaro merged commit dfc9b2a into tardis-sn:master Nov 20, 2024
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants