Better formatting for spacy train CLI
#2357
Conversation
Changed to use fixed-spaces rather than tabs to align table headers and data. ### Before: ``` Itn. P.Loss N.Loss UAS NER P. NER R. NER F. Tag % Token % 0 4618.857 2910.004 76.172 79.645 67.987 88.732 88.261 100.000 4436.9 6376.4 1 4671.972 3764.812 74.481 78.046 62.374 82.680 88.377 100.000 4672.2 6227.1 2 4742.756 3673.473 71.994 77.380 63.966 84.494 90.620 100.000 4298.0 5983.9 ``` ### After: ``` Itn. Dep Loss NER Loss UAS NER P. NER R. NER F. Tag % Token % CPU WPS GPU WPS 0 4618.857 2910.004 76.172 79.645 67.987 88.732 88.261 100.000 4436.9 6376.4 1 4671.972 3764.812 74.481 78.046 62.374 82.680 88.377 100.000 4672.2 6227.1 2 4742.756 3673.473 71.994 77.380 63.966 84.494 90.620 100.000 4298.0 5983.9 ```
|
Hi @BigstickCarpet, thanks for your pull request! 👍 It looks like you haven't filled in the spaCy Contributor Agreement (SCA) yet. The agrement ensures that we can use your contribution across the project. Once you've filled in the template, put it in the If you've already included the Contributor Agreement in your pull request above, you can ignore this message. |
ines
added
enhancement
There was a problem hiding this comment.
Ah, thanks – looks great! 👍 (I actually noticed the messed up formatting the other day and it bothered me, so it's really nice to have it look all consistent now.)
One small request: Could you move the contributor agreement to a file .github/contributors/BigstickCarpet.md?
|
@ines - Done. Sorry, I misunderstood the Contributor Agreement initially |
|
No worries! (And the contributor agreement thing is kind of a weird process – but it's the best we could come up with. Luckily, you'll only have to do it once!) |
Description
Changed to use fixed-spaces rather than tabs to align table headers and data.
Before:
After:
Types of change
Just display formatting
Checklist