Skip to content

Conversation

@felixcremer
Copy link
Collaborator

@felixcremer felixcremer commented Mar 19, 2025

This is to tidy up the code, so that we remove the unused code snippets.
Certain parts of the code are not removed, because they will be used for the next steps of the analysis.

I also removed some commented code.

@codecov
Copy link

codecov bot commented Mar 19, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.26%. Comparing base (b63c875) to head (a842fe0).
Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main      #98       +/-   ##
===========================================
+ Coverage   47.18%   61.26%   +14.07%     
===========================================
  Files           7        7               
  Lines         320      253       -67     
===========================================
+ Hits          151      155        +4     
+ Misses        169       98       -71     

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

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@felixcremer felixcremer changed the title Remove duplicated tau_recurrence function from analysis Tidy up code and move code for later analyses into separate files Mar 19, 2025
This function should be superseeded by the LazyAggDiskArray approach, because it can't deal with the aggregation of the data.
@felixcremer
Copy link
Collaborator Author

This is good to go for now. I don't know what is with the Documenter build. @danlooo could you have a look?

@danlooo
Copy link
Collaborator

danlooo commented Mar 19, 2025

Sorry, I installed an apt repo mixing debian and ubuntu. Fixed for now.

@danlooo danlooo merged commit c30421f into main Mar 19, 2025
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants