-
Notifications
You must be signed in to change notification settings - Fork 8
Tests #157
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
Tests #157
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## Torino #157 +/- ##
==========================================
+ Coverage 41.88% 43.53% +1.64%
==========================================
Files 47 49 +2
Lines 5567 5673 +106
==========================================
+ Hits 2332 2470 +138
+ Misses 3235 3203 -32
☔ View full report in Codecov by Sentry. |
@Elisa-Visentin is this supposed to be merged after #170 ? or can it be merged before? |
Here I am implementing some tests while we are upgrading the pipeline. This will be merged with/after the next pipeline PR (i.e., upgrades of the RFs, Dl2, DL3...). PR #170 is about semi-automatic scripts, so not dependent on tests. |
ok, I see. thanks! |
I'm now closing it since it is quite old. Maybe in the future I will add other tests to the pipeline (re-using some of this code), but, with the ongoing upgrades, this PR is now useless |
More tests for the 4 LSTs pipeline