Closed benfoley closed 3 years ago
When merged this will close #34
Should be a lint for that ;-D
On 23 Mar 2021, at 7:11 pm, Nicholas Lambourne @.***> wrote:
@nicklambourne approved this pull request.
One tiny nitpick, otherwise code looks good and have run/tested with no issues.
In elpis/gui/src/index.css https://github.com/CoEDL/elpis/pull/210#discussion_r599387390:
@@ -245,7 +245,9 @@ label.pad-right, color: #8abc1e; }
-.choose-dataset button { max-width: 300px } +.choose-dataset button {
- max-width: 300px indent should be 4 spaces, not 2
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/CoEDL/elpis/pull/210#pullrequestreview-618351065, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABLD5GFHMP63AIERGHZ5YG3TFBLLJANCNFSM4ZSVVVBA.
This gets training results after training is done, and includes them in the GUI model table.
Plus a couple of other fixes.