Closed atharva-2001 closed 2 days ago
*beep* *bop* Hi human, I ran ruff on the latest commit (8b6fda302664b83f6dacc5d2b1f0b16a337ff49a). Here are the outputs produced. Results can also be downloaded as artifacts here. Summarised output:
Complete output(might be large):
Check out this pull request on
See visual diffs & provide feedback on Jupyter Notebooks.
Powered by ReviewNB
Can you fix the benchmarks? Docs as well.
*beep* *bop* Hi human, I ran benchmarks as you asked comparing master (260207cb7cee6829390d77fbf48de53843818f09) and the latest commit (8b6fda302664b83f6dacc5d2b1f0b16a337ff49a). Here are the logs produced by ASV. Results can also be downloaded as artifacts here.
Significantly changed benchmarks:
All benchmarks:
If you want to see the graph of the results, you can check it here
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 70.79%. Comparing base (
be4ec9a
) to head (8b6fda3
). Report is 3 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Do you know why docs are failing?
I think it is because of the YAML file not being updated because it is a pull request target event. The YAML file was picked up from the master branch instead of this branch.
On Fri, 6 Sep 2024 at 7:46 PM, Andrew @.***> wrote:
Do you know why docs are failing?
— Reply to this email directly, view it on GitHub https://github.com/tardis-sn/tardis/pull/2808#issuecomment-2334162066, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANKOCXAKNDMI54URPKGDUQDZVG2LJAVCNFSM6AAAAABM7N3M7GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZUGE3DEMBWGY . You are receiving this because you authored the thread.Message ID: @.***>
I think it should all work once the PR is merged!
On Fri, 6 Sep 2024 at 10:21 PM, Atharva Arya @.***> wrote:
I think it is because of the YAML file not being updated because it is a pull request target event. The YAML file was picked up from the master branch instead of this branch.
On Fri, 6 Sep 2024 at 7:46 PM, Andrew @.***> wrote:
Do you know why docs are failing?
— Reply to this email directly, view it on GitHub https://github.com/tardis-sn/tardis/pull/2808#issuecomment-2334162066, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANKOCXAKNDMI54URPKGDUQDZVG2LJAVCNFSM6AAAAABM7N3M7GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZUGE3DEMBWGY . You are receiving this because you authored the thread.Message ID: @.***>
:pencil: Description
Type: :biohazard:
breaking change
| :vertical_traffic_light:testing
| :roller_coaster:infrastructure
This PR aims to remove references of refdata from the tardis repo. https://github.com/tardis-sn/tardis-regression-data/pull/21 and https://github.com/tardis-sn/tardis-regression-data/pull/22 will need to be merged before in order for tests to pass.
:pushpin: Resources
Examples, notebooks, and links to useful references.
:vertical_traffic_light: Testing
How did you test these changes?
:ballot_box_with_check: Checklist
build_docs
label