EducationalTestingService / skll

SciKit-Learn Laboratory (SKLL) makes it easy to run machine learning experiments.
http://skll.readthedocs.org
Other
551 stars 67 forks source link

Black #652

Closed mulhod closed 1 year ago

mulhod commented 3 years ago

Just trying out black. Please leave feedback.

This is a pre-commit hook, by the way.

codecov[bot] commented 3 years ago

Codecov Report

Merging #652 (47319b9) into main (32823c0) will not change coverage. The diff coverage is 93.43%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #652   +/-   ##
=======================================
  Coverage   95.06%   95.06%           
=======================================
  Files          27       27           
  Lines        3098     3098           
=======================================
  Hits         2945     2945           
  Misses        153      153           
Impacted Files Coverage Δ
skll/data/readers.py 90.13% <83.82%> (ø)
skll/experiments/utils.py 93.45% <86.11%> (ø)
skll/data/writers.py 94.11% <89.28%> (ø)
skll/config/__init__.py 95.09% <89.65%> (ø)
skll/experiments/__init__.py 95.19% <92.00%> (ø)
skll/learner/__init__.py 96.28% <93.12%> (ø)
skll/utils/commandline/skll_convert.py 93.05% <93.75%> (ø)
skll/data/featureset.py 91.13% <94.28%> (ø)
...utils/commandline/compute_eval_from_predictions.py 97.18% <94.73%> (ø)
skll/learner/utils.py 94.33% <96.36%> (ø)
... and 17 more

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 32823c0...47319b9. Read the comment docs.

desilinguist commented 1 year ago

I am going to close this and start a fresh PR from main since there are too many merge conflicts.