sepandhaghighi / pycm

Multi-class confusion matrix library in Python
http://pycm.io
MIT License
1.45k stars 124 forks source link

Markdown Html Mixture #506

Closed sadrasabouri closed 1 year ago

sadrasabouri commented 1 year ago

Reference Issues/PRs

496

What does this implement/fix? Explain your changes.

Minor edits are applied after merging this PR:

codecov[bot] commented 1 year ago

Codecov Report

Merging #506 (c7c2c17) into dev (626aac9) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #506   +/-   ##
=======================================
  Coverage   99.31%   99.31%           
=======================================
  Files          13       13           
  Lines        3031     3031           
  Branches      451      451           
=======================================
  Hits         3010     3010           
  Misses          8        8           
  Partials       13       13           
sadrasabouri commented 1 year ago

@sadrasabouri Thanks for your efforts 100

* `$|C|$` is a LateX formula so please revert this one.

* Some sections were missed, am I right?

image

You were right. I was using a mistaken regular expression for checking their existence. I updated it to the below code and now we hunted all of them:

sadra@beta:~/local/pycm$ grep "Notice" Document/Document.ipynb | grep "\`"
    "    <li><span style=\"color:red;\">Notice </span> : `matrix`, `normalized_matrix` & `normalized_table` added in <span style=\"color:red;\">version 1.5</span> (changed from print style)</li>\n",
    "    <li><span style=\"color:red;\">Notice </span> : `classes` added in <span style=\"color:red;\">version 3.2</span></li>\n",

They are fixed now and it's ready for your review.