Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Home page - update UI and sort by network outputs #790

Merged
merged 1 commit into from
Jun 2, 2016

Conversation

jmancewicz
Copy link
Contributor

I've made a few changes to the front page tables. I've moved the running jobs to the angularjs table, and ported the selection into angular from jquery.

This also has the settings menu which allows the user to modify the view of the model table. These settings are persistent.

screen shot 2016-05-25 at 5 35 46 pm

@@ -0,0 +1,233 @@
(function (root, factory) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we use a minimized version of this file and add a copyright notice to the top? You didn't edit it, did you?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@lukeyeager
Copy link
Member

lukeyeager commented May 28, 2016

There's a bug in the sorting by network outputs. When I try to sort based on a network output that only exists for one model, all the models that don't have that network output show up at the top, regardless of whether I sort ascending or descending. I'd expect anything with a non-null value at the top, sorted by the requested method.

home-pages-bug1

More sorting weirdness:

home-page-bug3

@lukeyeager
Copy link
Member

When sorting by "Loss last", is it using the training output or the validation output? Both combined?

@lukeyeager
Copy link
Member

I love that finished jobs drop down into the appropriate panel (solving #741 (comment)).

However, it's missing most of the job metadata when it drops down like elapsed time and network outputs:

home-pages-bug2

@lukeyeager
Copy link
Member

Can we not rename the network outputs? So, leave them like "loss_bbox" instead of "Loss Bbox".

@jmancewicz
Copy link
Contributor Author

Initially, I had total control of the column titles and the data keys. The keys were all lowercase with underscores and the titles were title case with spaces. So they could be converted back and forth. Now with data keys such as accuracyA, that assumption was no longer valid. The column sorting converted the titles to data keys, and the attempt to sort 'accuracyA' was treated as 'accuracya' and not sorting. That's why the sort had the blanks at the top; it didn't have any field named accuracya.

And I've changed the method to not capitalize a word if the word has any capital letters in it. 'xyz_last' => 'Xyz Last', but 'xyZ_last' => 'xyZ last'.

I'm made it so that field names are only modified for display. That fixed the problem for my test. @lukeyeager, please verify that that fix sorts your accuracyA column correctly.

@jmancewicz
Copy link
Contributor Author

Again mAP_last was displaying as Map Last and converted to map_last as a key. Now, it use mAP_last as the key and display as 'mAP Last'.

@jmancewicz
Copy link
Contributor Author

Does the conversion of "loss_bbox" to "Loss Bbox" make it confusing?

@jmancewicz
Copy link
Contributor Author

The loss would be the validation loss.

In the /models page, the loss would have been the training loss.

I've added a commit in which all outputs are there. I don't like that the names need to be so long, and there's data that I would think we could skip like 'min epoch'.

screen shot 2016-05-29 at 1 25 24 pm

I've removed the conversion of the keys (from 'leaning_rate' to 'Learning Rate'). I dislike that it adds inconsistency in the case of ui elements, but do like that the names are consistent with how they are represented elsewhere. So, I'm good either way.

I've added (train) and (val) to the labels which how they are represented on the graphs.

@lukeyeager lukeyeager changed the title Front Page Tables Update [Not ready for merge. Need to rebase] Home page - update UI and sort by network outputs May 31, 2016
@lukeyeager
Copy link
Member

I like the flexibility of the new list - now I have access to everything. However, I have 203 things I can sort by (calculated with $("ul.dropdown-menu > li").length), which makes the dropdown list longer than my list of 170 jobs. That seems excessive.

Also, the sorting still isn't working. See screenshots above.

@lukeyeager lukeyeager self-assigned this Jun 2, 2016
@jmancewicz
Copy link
Contributor Author

Just committed a change that fixes the sorting issue caused by spaces in the field name.

@lukeyeager
Copy link
Member

Confirmed that sorting works now

@jmancewicz
Copy link
Contributor Author

Added a filter to move the empty values to the end of the list during sort.

@jmancewicz
Copy link
Contributor Author

Changed the page so that when the status change causes the job to move from the Running Jobs table, it pulls date from the server for the job including all the network output data that isn't needed for the Running Jobs view.

@lukeyeager
Copy link
Member

Looks good to me! Thanks for the fix - this is really slick now.

Please squash and I'll merge.

@lukeyeager lukeyeager merged commit dab8426 into NVIDIA:master Jun 2, 2016
@jmancewicz jmancewicz deleted the ng-models-page branch June 2, 2016 22:06
@lukeyeager lukeyeager mentioned this pull request Oct 31, 2016
SlipknotTN pushed a commit to cynnyx/DIGITS that referenced this pull request Mar 30, 2017
Home page - update UI and sort by network outputs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants