Closed KasukabeDefenceForce closed 3 weeks ago
Check out this pull request on
See visual diffs & provide feedback on Jupyter Notebooks.
Powered by ReviewNB
*beep* *bop* Hi human, I ran ruff on the latest commit (bf40f9420eb99238617f66a377ce871e7647b4d8). Here are the outputs produced. Results can also be downloaded as artifacts here. Summarised output:
Complete output(might be large):
I prefer to use requests over the ads package because the former is much faster. I could fetch all the papers in approx 2.5 minutes using requests while it took approx 7 minutes to do the same using ads.
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 69.37%. Comparing base (
8a317dd
) to head (bf40f94
). Report is 8 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Sample Workflow Run: https://github.com/KasukabeDefenceForce/tardis/actions/runs/10387669495
Sample PR: https://github.com/KasukabeDefenceForce/tardis/pull/44
*beep* *bop*
Hi, human.
The docs
workflow has succeeded :heavy_check_mark:
Click here to see your results.
*beep* *bop* Hi human, I ran benchmarks as you asked comparing master (a054d743a0b35fd153e407a2f5df973b103bd32e) and the latest commit (bf40f9420eb99238617f66a377ce871e7647b4d8). 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
:pencil: Description
Type: :beetle:
bugfix
| :rocket:feature
| :biohazard:breaking change
| :vertical_traffic_light:testing
| :memo:documentation
| :roller_coaster:infrastructure
This PR aims at improving the logic for fetching the research papers which use Tardis and updating them in the docs. Old Logic: We used to search for "Tardis" and "Kerzendorf" or "tardis" and "supernova" in the full text of the article and were published in the NATUR publication. The total number of articles with this logic were 50. New Logic: We fetch all the papers which have cited this article. The total number of articles fetched with this logic are 132. You can have a look at these here
: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