ponder-lab / Imperative-DL-Study-Web-App

0 stars 0 forks source link

Mouseover categories with their descriptions #73

Closed khatchad closed 3 years ago

khatchad commented 3 years ago

For each of the category columns associated with categorizations (in both the bug fixes and the categorization views), if you hover your mouse over a category, could you display its associated description? This happens currently in phpMyAdmin. Here is an example. It seems very easy to do but you'll need to query for the category description (or alter the original query to join the category tables, which would be more efficient to do).

mZneit commented 3 years ago

This worked for problem category and problem cause columns in the Categorizations table. Currently the problem symptoms and problem fixes do not have description columns in models.py. I added the descriptions locally and the mouseover functionality worked for the problem_fixes. For problem_symptoms there's still a problem that needs to be fixed.

mZneit commented 3 years ago

@tatianacv the problem_symptoms table in the db doesn't have a description column, actually we need this column when hovering the mouse over a column cell to display its description.

ID Symptom
1 None
2 Lower performance
3 Missing variable value.
4 Errors-in-exception-handling
5 No support for eager mode
6 Dynamic features unavailable.
7 Autograph warning
324 new symptom
334 test
344 ponder
354 23
364 This is a new symptom
374 gsed
mZneit commented 3 years ago

The problem_fixes table already has a description column.

ID Fix Description
1 Remove tf.function decorator.
2 Use tf.function.
3 Re-add tf.function decorator.
4 Add state as function parameter.
5 Use of tf.eagerly_executing() to check whether we are in graph mode
6 Embed converted function declaration in calling function. This way, the Python function being converted by tf.function() (transitively, in some cases) now has access to the callers variables, i.e., they are now in-scope statically where they may have been in scope dynamically before.
7 None
354 new fix
364 Aaaaa
374 ponder
384 23
394 This a new fix
khatchad commented 3 years ago

Right. These columns need to be added. Thanks!

tatianacv commented 3 years ago

@tatianacv the problem_symptoms table in the db doesn't have a description column, actually we need this column when hovering the mouse over a column cell to display its description. ID Symptom 1 None 2 Lower performance 3 Missing variable value. 4 Errors-in-exception-handling 5 No support for eager mode 6 Dynamic features unavailable. 7 Autograph warning 324 new symptom 334 test 344 ponder 354 23 364 This is a new symptom 374 gsed

It has it now.

mZneit commented 3 years ago

Commit 465f076a9bc45e85fc03f45104a5cf29ab9023df

khatchad commented 3 years ago

Thanks!