Skip to content

Conversation

@dependabot
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Aug 12, 2025

Bumps codecov/codecov-action from 4 to 5.

Release notes

Sourced from codecov/codecov-action's releases.

v5.0.0

v5 Release

v5 of the Codecov GitHub Action will use the Codecov Wrapper to encapsulate the CLI. This will help ensure that the Action gets updates quicker.

Migration Guide

The v5 release also coincides with the opt-out feature for tokens for public repositories. In the Global Upload Token section of the settings page of an organization in codecov.io, you can set the ability for Codecov to receive a coverage reports from any source. This will allow contributors or other members of a repository to upload without needing access to the Codecov token. For more details see how to upload without a token.

[!WARNING]
The following arguments have been changed

  • file (this has been deprecated in favor of files)
  • plugin (this has been deprecated in favor of plugins)

The following arguments have been added:

  • binary
  • gcov_args
  • gcov_executable
  • gcov_ignore
  • gcov_include
  • report_type
  • skip_validation
  • swift_project

You can see their usage in the action.yml file.

What's Changed

... (truncated)

Changelog

Sourced from codecov/codecov-action's changelog.

v5 Release

v5 of the Codecov GitHub Action will use the Codecov Wrapper to encapsulate the CLI. This will help ensure that the Action gets updates quicker.

Migration Guide

The v5 release also coincides with the opt-out feature for tokens for public repositories. In the Global Upload Token section of the settings page of an organization in codecov.io, you can set the ability for Codecov to receive a coverage reports from any source. This will allow contributors or other members of a repository to upload without needing access to the Codecov token. For more details see how to upload without a token.

[!WARNING] The following arguments have been changed

  • file (this has been deprecated in favor of files)
  • plugin (this has been deprecated in favor of plugins)

The following arguments have been added:

  • binary
  • gcov_args
  • gcov_executable
  • gcov_ignore
  • gcov_include
  • report_type
  • skip_validation
  • swift_project

You can see their usage in the action.yml file.

What's Changed

... (truncated)

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action) from 4 to 5.
- [Release notes](https://github.com/codecov/codecov-action/releases)
- [Changelog](https://github.com/codecov/codecov-action/blob/main/CHANGELOG.md)
- [Commits](codecov/codecov-action@v4...v5)

---
updated-dependencies:
- dependency-name: codecov/codecov-action
  dependency-version: '5'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot added dependencies Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code labels Aug 12, 2025
@tardis-bot
Copy link
Contributor

*beep* *bop*

Hi, human.

I'm the @tardis-bot and I noticed that your email is not associated with an ORCID ID in our database.

Please add your email and ORCID ID to the .orcid.csv file in your current branch and push the changes to this pull request.

If you don't have an ORCID ID yet, you can create one for free at orcid.org. ORCID IDs help ensure you get proper credit for your scientific contributions.

The format should be:

email,orcid
[email protected],0000-0000-0000-0000

@tardis-bot
Copy link
Contributor

*beep* *bop*

Hi, human.

I'm the @tardis-bot and couldn't find your records in my database. I think we don't know each other, or you changed your credentials recently.

Please add your name and email to .mailmap in your current branch and push the changes to this pull request.

In case you need to map an existing alias, follow this example.

@tardis-bot
Copy link
Contributor

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

Details
273	       	[ ] syntax-error
38	W293   	[ ] blank-line-with-whitespace
13	G004   	[ ] logging-f-string
12	F401   	[ ] unused-import
8	I001   	[*] unsorted-imports
6	E701   	[ ] multiple-statements-on-one-line-colon
5	E402   	[ ] module-import-not-at-top-of-file
5	F405   	[ ] undefined-local-with-import-star-usage
4	D204   	[*] incorrect-blank-line-after-class
4	F811   	[ ] redefined-while-unused
4	UP015  	[*] redundant-open-modes
3	E703   	[ ] useless-semicolon
3	E902   	[ ] io-error
3	F821   	[ ] undefined-name
3	RET505 	[*] superfluous-else-return
2	W291   	[ ] trailing-whitespace
1	E712   	[ ] true-false-comparison
1	E722   	[ ] bare-except
1	NPY201 	[ ] numpy2-deprecation
1	PGH004 	[ ] blanket-noqa
1	PLW0127	[ ] self-assigning-variable
1	RET506 	[*] superfluous-else-raise
1	RUF015 	[ ] unnecessary-iterable-allocation-for-first-element
1	RUF021 	[*] parenthesize-chained-operators
1	TRY300 	[ ] try-consider-else
1	UP009  	[*] utf8-encoding-declaration
1	UP030  	[ ] format-literals
1	UP032  	[*] f-string
Found 398 errors.
[*] 41 fixable with the `--fix` option (30 hidden fixes can be enabled with the `--unsafe-fixes` option).

Complete output(might be large):

Details
docs/conf.py:1:1: UP009 [*] UTF-8 encoding declaration is unnecessary
docs/conf.py:27:1: I001 [*] Import block is un-sorted or un-formatted
docs/conf.py:30:8: F401 [*] `tardis` imported but unused
docs/conf.py:37:43: PGH004 Use specific rule codes when using `noqa`
docs/conf.py:47:27: UP015 [*] Unnecessary mode argument
docs/conf.py:68:1: F405 `exclude_patterns` may be undefined, or defined from star imports
docs/conf.py:69:1: F405 `exclude_patterns` may be undefined, or defined from star imports
docs/conf.py:70:1: F405 `exclude_patterns` may be undefined, or defined from star imports
docs/conf.py:71:1: F405 `exclude_patterns` may be undefined, or defined from star imports
docs/conf.py:72:1: F405 `exclude_patterns` may be undefined, or defined from star imports
docs/conf.py:142:1: W293 Blank line contains whitespace
docs/conf.py:177:1: W293 Blank line contains whitespace
docs/conf.py:194:13: UP030 Use implicit references for positional format fields
docs/conf.py:194:13: UP032 [*] Use f-string instead of `format` call
docs/conf.py:221:1: E402 Module level import not at top of file
docs/conf.py:293:4: E712 Avoid equality comparisons to `True`; use `toml_config_tool_dict["tardis"]['edit_on_github']:` for truth checks
docs/conf.py:315:1: E402 Module level import not at top of file
docs/conf.py:367:1: E402 Module level import not at top of file
docs/conf.py:382:37: UP015 [*] Unnecessary modes, use `w`
docs/conf.py:397:41: UP015 [*] Unnecessary modes, use `w`
docs/conf.py:412:37: UP015 [*] Unnecessary modes, use `w`
tardis/base.py:106:13: PLW0127 Self-assignment of variable `atom_data`
tardis/io/logger/logger.py:1:1: I001 [*] Import block is un-sorted or un-formatted
tardis/io/logger/logger.py:8:23: F401 [*] `functools.lru_cache` imported but unused
tardis/io/logger/logger.py:23:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:38:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:43:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:79:5: D204 [*] 1 blank line required after class docstring
tardis/io/logger/logger.py:80:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:116:5: D204 [*] 1 blank line required after class docstring
tardis/io/logger/logger.py:117:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:119:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:148:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:170:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:175:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:186:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:193:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:208:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:212:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:241:1: W293 [*] Blank line contains whitespace
tardis/io/logger/logger.py:253:5: D204 [*] 1 blank line required after class docstring
tardis/io/logger/logger.py:254:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:256:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:275:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:284:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:299:21: G004 Logging statement uses f-string
tardis/io/logger/logger.py:361:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:373:1: W293 [*] Blank line contains whitespace
tardis/io/logger/logger.py:380:1: W293 [*] Blank line contains whitespace
tardis/io/logger/logger.py:386:1: W293 [*] Blank line contains whitespace
tardis/io/logger/logger.py:391:5: D204 [*] 1 blank line required after class docstring
tardis/io/logger/logger.py:392:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:403:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:408:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:418:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:421:1: W293 Blank line contains whitespace
tardis/io/logger/logger.py:432:1: W293 Blank line contains whitespace
tardis/io/logger/logger_widget.py:1:1: E902 No such file or directory (os error 2)
tardis/simulation/base.py:1:1: I001 [*] Import block is un-sorted or un-formatted
tardis/simulation/base.py:245:17: G004 Logging statement uses f-string
tardis/simulation/base.py:418:13: G004 Logging statement uses f-string
tardis/simulation/base.py:519:13: G004 Logging statement uses f-string
tardis/simulation/base.py:605:13: G004 Logging statement uses f-string
tardis/simulation/base.py:610:13: G004 Logging statement uses f-string
tardis/simulation/base.py:661:13: TRY300 Consider moving this statement to an `else` block
tardis/simulation/base.py:663:26: G004 Logging statement uses f-string
tardis/util/base.py:184:20: NPY201 `np.trapz` will be removed in NumPy 2.0. Use `numpy.trapezoid` on NumPy 2.0, or ignore this warning on earlier versions.
tardis/util/base.py:335:5: RET505 [*] Unnecessary `else` after `return` statement
tardis/util/base.py:509:5: E722 Do not use bare `except`
tardis/util/base.py:617:5: RET505 [*] Unnecessary `elif` after `return` statement
tardis/util/environment.py:1:1: E902 No such file or directory (os error 2)
tardis/util/panel_init.py:1:1: E902 No such file or directory (os error 2)
tardis/visualization/__init__.py:3:57: F401 `tardis.visualization.tools.convergence_plot.ConvergencePlots` imported but unused; consider removing, adding to `__all__`, or using a redundant alias
tardis/visualization/__init__.py:4:49: F401 `tardis.visualization.tools.liv_plot.LIVPlotter` imported but unused; consider removing, adding to `__all__`, or using a redundant alias
tardis/visualization/__init__.py:5:53: F401 `tardis.visualization.tools.rpacket_plot.RPacketPlotter` imported but unused; consider removing, adding to `__all__`, or using a redundant alias
tardis/visualization/__init__.py:6:50: F401 `tardis.visualization.tools.sdec_plot.SDECPlotter` imported but unused; consider removing, adding to `__all__`, or using a redundant alias
tardis/visualization/__init__.py:7:59: F401 `tardis.visualization.widgets.custom_abundance.CustomAbundanceWidget` imported but unused; consider removing, adding to `__all__`, or using a redundant alias
tardis/visualization/__init__.py:8:51: F401 `tardis.visualization.widgets.grotrian.GrotrianWidget` imported but unused; consider removing, adding to `__all__`, or using a redundant alias
tardis/visualization/__init__.py:9:52: F401 `tardis.visualization.widgets.line_info.LineInfoWidget` imported but unused; consider removing, adding to `__all__`, or using a redundant alias
tardis/visualization/__init__.py:11:5: F401 `tardis.visualization.widgets.shell_info.shell_info_from_hdf` imported but unused; consider removing, adding to `__all__`, or using a redundant alias
tardis/visualization/__init__.py:12:5: F401 `tardis.visualization.widgets.shell_info.shell_info_from_simulation` imported but unused; consider removing, adding to `__all__`, or using a redundant alias
tardis/visualization/tools/tests/test_convergence_plot.py:3:1: I001 [*] Import block is un-sorted or un-formatted
tardis/visualization/tools/tests/test_convergence_plot.py:14:25: F811 [*] Redefinition of unused `defaultdict` from line 3
tardis/visualization/tools/tests/test_convergence_plot.py:15:32: F811 [*] Redefinition of unused `go` from line 6
tardis/visualization/tools/tests/test_convergence_plot.py:16:30: F811 [*] Redefinition of unused `u` from line 8
tardis/visualization/widgets/custom_abundance.py:737:16: RUF021 [*] Parenthesize `a and b` expressions when chaining `and` and `or` together, to make the precedence clear
tardis/visualization/widgets/custom_abundance.py:1127:9: RET505 [*] Unnecessary `else` after `return` statement
tardis/visualization/widgets/custom_abundance.py:1418:9: RET506 [*] Unnecessary `else` after `raise` statement
tardis/visualization/widgets/custom_abundance.py:1721:9: F811 Redefinition of unused `input_d_time_0_eventhandler` from line 1710
tardis/visualization/widgets/custom_abundance.py:1751:13: F821 Undefined name `display`
tardis/visualization/widgets/custom_abundance.py:1753:13: F821 Undefined name `display`
tardis/visualization/widgets/custom_abundance.py:1755:13: F821 Undefined name `display`
tardis/visualization/widgets/line_info.py:3:1: I001 [*] Import block is un-sorted or un-formatted
tardis/visualization/widgets/line_info.py:450:1: W293 [*] Blank line contains whitespace
tardis/visualization/widgets/line_info.py:454:1: W293 [*] Blank line contains whitespace
tardis/visualization/widgets/line_info.py:458:1: W293 [*] Blank line contains whitespace
tardis/visualization/widgets/line_info.py:462:1: W293 [*] Blank line contains whitespace
tardis/visualization/widgets/line_info.py:467:1: W293 [*] Blank line contains whitespace
tardis/visualization/widgets/line_info.py:476:1: W293 [*] Blank line contains whitespace
tardis/visualization/widgets/line_info.py:531:1: W293 [*] Blank line contains whitespace
tardis/visualization/widgets/line_info.py:715:1: W293 [*] Blank line contains whitespace
tardis/visualization/widgets/tests/test_custom_abundance.py:224:37: RUF015 Prefer `next(iter(unique_v))` over single element slice
tardis/visualization/widgets/tests/test_line_info.py:9:1: E402 Module level import not at top of file
tardis/visualization/widgets/tests/test_shell_info.py:6:1: E402 Module level import not at top of file
tardis/workflows/simple_tardis_workflow.py:1:1: I001 [*] Import block is un-sorted or un-formatted
tardis/workflows/simple_tardis_workflow.py:249:17: G004 Logging statement uses f-string
tardis/workflows/simple_tardis_workflow.py:470:17: G004 Logging statement uses f-string
tardis/workflows/standard_tardis_workflow.py:170:13: G004 Logging statement uses f-string
tardis/workflows/standard_tardis_workflow.py:210:17: G004 Logging statement uses f-string
tardis/workflows/workflow_logging.py:69:13: G004 Logging statement uses f-string
tardis/workflows/workflow_logging.py:107:21: G004 Logging statement uses f-string
Found 111 errors.
[*] 38 fixable with the `--fix` option (30 hidden fixes can be enabled with the `--unsafe-fixes` option).

@tardis-bot
Copy link
Contributor

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

Significantly changed benchmarks:

Details

All benchmarks:

Details
Benchmarks that have stayed the same:

| Change   | Before [27150719] <master>   | After [aee52304]    | Ratio   | Benchmark (Parameter)                                                                                                               |
|----------|------------------------------|---------------------|---------|-------------------------------------------------------------------------------------------------------------------------------------|
|          | 1.30±0.3μs                   | 1.61±0.3μs          | ~1.24   | transport_geometry_calculate_distances.BenchmarkTransportGeometryCalculateDistances.time_calculate_distance_line                    |
|          | 45.8±30μs                    | 50.6±30μs           | ~1.11   | transport_montecarlo_interaction.BenchmarkTransportMontecarloInteraction.time_line_scatter                                          |
|          | 39.3±0.07μs                  | 43.6±0.05μs         | ~1.11   | transport_montecarlo_packet_trackers.BenchmarkTransportMontecarloPacketTrackers.time_generate_rpacket_tracker_list                  |
|          | 2.75±0.3ms                   | 3.05±0.5ms          | ~1.11   | transport_montecarlo_single_packet_loop.BenchmarkTransportMontecarloSinglePacketLoop.time_single_packet_loop                        |
|          | 541±200ns                    | 481±200ns           | ~0.89   | opacities_opacity.BenchmarkMontecarloMontecarloNumbaOpacities.time_photoabsorption_opacity_calculation                              |
|          | 45.9±30μs                    | 40.8±30μs           | ~0.89   | transport_montecarlo_interaction.BenchmarkTransportMontecarloInteraction.time_line_emission                                         |
|          | 3.14±0.4μs                   | 2.64±0.5μs          | ~0.84   | transport_montecarlo_vpacket.BenchmarkMontecarloMontecarloNumbaVpacket.time_trace_vpacket_within_shell                              |
|          | 701±100ns                    | 570±100ns           | ~0.81   | opacities_opacity.BenchmarkMontecarloMontecarloNumbaOpacities.time_pair_creation_opacity_calculation                                |
|          | 26.4±7μs                     | 21.4±5μs            | ~0.81   | transport_montecarlo_packet_trackers.BenchmarkTransportMontecarloPacketTrackers.time_generate_rpacket_last_interaction_tracker_list |
|          | 1.92±2μs                     | 1.97±1μs            | 1.03    | transport_montecarlo_estimators_radfield_estimator_calcs.BenchmarkMontecarloMontecarloNumbaPacket.time_update_line_estimators       |
|          | 3.04±0.3μs                   | 3.12±0.4μs          | 1.03    | transport_montecarlo_vpacket.BenchmarkMontecarloMontecarloNumbaVpacket.time_trace_bad_vpacket                                       |
|          | 4.52±0.03ms                  | 4.61±0.07ms         | 1.02    | opacities_opacity_state.BenchmarkOpacitiesOpacityState.time_opacity_state_initialize('macroatom')                                   |
|          | 46.2±0.02s                   | 46.4±0.02s          | 1.00    | spectrum_formal_integral.BenchmarkTransportMontecarloFormalIntegral.time_FormalIntegrator_functions                                 |
|          | 61.1±0.5ms                   | 61.3±0.08ms         | 1.00    | transport_montecarlo_packet_trackers.BenchmarkTransportMontecarloPacketTrackers.time_rpacket_trackers_to_dataframe                  |
|          | 1.02±0m                      | 1.01±0m             | 0.99    | run_tardis.BenchmarkRunTardis.time_run_tardis_rpacket_tracking                                                                      |
|          | 7.07±2μs                     | 6.99±2μs            | 0.99    | transport_montecarlo_vpacket.BenchmarkMontecarloMontecarloNumbaVpacket.time_trace_vpacket_volley                                    |
|          | 1.17±0μs                     | 1.13±0μs            | 0.97    | transport_geometry_calculate_distances.BenchmarkTransportGeometryCalculateDistances.time_calculate_distance_boundary                |
|          | 521±200ns                    | 501±200ns           | 0.96    | opacities_opacity.BenchmarkMontecarloMontecarloNumbaOpacities.time_compton_opacity_calculation                                      |
|          | 5.76±1μs                     | 5.54±1μs            | 0.96    | transport_montecarlo_vpacket.BenchmarkMontecarloMontecarloNumbaVpacket.time_trace_vpacket                                           |
|          | 39.8±0.09s                   | 37.9±0.1s           | 0.95    | run_tardis.BenchmarkRunTardis.time_run_tardis                                                                                       |
|          | 2.85±0.03ms                  | 2.66±0.01ms         | 0.93    | opacities_opacity_state.BenchmarkOpacitiesOpacityState.time_opacity_state_initialize('scatter')                                     |
|          | 209±0.07ns                   | 194±0.3ns           | 0.93    | spectrum_formal_integral.BenchmarkTransportMontecarloFormalIntegral.time_intensity_black_body                                       |
|          | 743±0.9ns                    | 692±0.2ns           | 0.93    | transport_montecarlo_interaction.BenchmarkTransportMontecarloInteraction.time_thomson_scatter                                       |
|          | 2.78±0.03ms                  | 2.58±0.03ms         | 0.92    | transport_montecarlo_main_loop.BenchmarkTransportMontecarloMontecarloMainLoop.time_montecarlo_main_loop                             |

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

@andrewfullard andrewfullard merged commit f6fa09c into master Aug 13, 2025
33 of 42 checks passed
@dependabot dependabot bot deleted the dependabot/github_actions/codecov/codecov-action-5 branch August 13, 2025 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants