-
Notifications
You must be signed in to change notification settings - Fork 9
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
In_out_versions #252
In_out_versions #252
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## auto_MCP_DL2_DL3 #252 +/- ##
=================================================
Coverage 77.23% 77.23%
=================================================
Files 21 21
Lines 2614 2614
=================================================
Hits 2019 2019
Misses 595 595 ☔ View full report in Codecov by Sentry. |
I checked the files and the changes look ok. Once a test is run we should also check the job_accounting script if it is not missing any input files because they are in a different folder |
I am still testing it (IT queues are a bit stuck) but I will check both output and, as you suggested, job_accounting. Hopefully during the we... |
Actually there was a small issue with directories (I have to create the directory with the current version name to be able to store the config. file for stereo and coincidence, or the scripts cannot read it). Now it should be fixed |
As for job accounting, I plan to fix and check it today :) Edit: seems fine! |
looks good, shall we merge? |
For me, yes (I finished all the tests and job accounting is fine) |
Add the possibility to use, for each step, input data processed with an older MCP version wrt the current one