Closed cl4yton closed 2 years ago
Merging #301 (2e153f4) into master (6fd949a) will decrease coverage by
0.00%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #301 +/- ##
==========================================
- Coverage 66.90% 66.89% -0.01%
==========================================
Files 106 106
Lines 21725 21725
==========================================
- Hits 14536 14534 -2
- Misses 7189 7191 +2
Impacted Files | Coverage Δ | |
---|---|---|
automates/model_analysis/graph_manipulation.py | 70.19% <0.00%> (-0.56%) |
:arrow_down: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 6fd949a...2e153f4. Read the comment docs.
Code cov went down a minuscule amount for automates/model_analysis/graph_manipulation.py
Hi @titomeister : This has been a task I have meant to get to for a long time. Dependabot has identified several of our requirements.txt module versions that are at older versions that should be increased. This PR makes the following changes to requirements.txt:
nltk is only used by scripts/code_summarization , which is older code created by Paul Hein. I've added to the README here that this code may need to be verified, but I'm ok bumping without testing for now.
I'm pretty sure both of the notebook and numpy changes will not cause problems, but could you please run our PA pipeline and its tests to verify?