ModelInference / perfume-frontend

Web-based frontend for Perfume tool
Other
1 stars 0 forks source link

Add logs from the demo video to the examples #72

Closed bestchai closed 4 years ago

bestchai commented 4 years ago

What:

[Issue created by stanleyrya: 2015-10-20] [Last updated on bitbucket: 2015-12-22]

[Comment created by ohmann: 2015-12-22] Merged in: e8f2c480b36b

(Note that this is using 2--the unshuffled large log--because 3 causes the frontend to hang. This is documented in issue #100.)

[Comment created by ohmann: 2015-12-15] This gave me the idea for #98, which would be a very useful feature.

It also made me notice that when using the shuffled log, inference never seems to terminate in the frontend. Running the backend manually infers the model without issue.

[Comment created by ohmann: 2015-12-15] I clock it at about 7-10 seconds for (2), and (3) should take almost exactly as long.

[Comment created by brunyuriy: 2015-12-15] I think that's OK. It's the slow log. It's fine.

[Comment created by brunyuriy: 2015-12-15] I think 3. Shuffled large.

How long does it take to infer a model?

[Comment created by ohmann: 2015-12-15] Looks good, but one question for all (@brunyuriy @bestchai):

Which version of the djQ log do we want?

  1. Unshuffled small (from the assignment)
  2. Unshuffled large (from the video)
  3. Shuffled large (from the paper screenshot)

This branch currently contains (2). I'd lean towards (3), except that it takes a while to infer the model for (2) or (3). Therefore, I vote (1).

[Comment created by stanleyrya: 2015-12-15] Now includes the new dropdown button!

837ad562fbd3

[Comment created by ohmann: 2015-12-16] @bestchai, I had the same idea, and that's what issue #98 is. :)

[Comment created by bestchai: 2015-12-16] I think slow log is good to have -- it shows the kind of long log that Perfume can handle.

However, 7-10 is long, so I think we need a new issue (since I don't think this exists yet) to have an indicator (e.g., spinning wheel) that is animated after the user presses the button to process the log (the button should also be disabled).

[Comment created by stanleyrya: 2015-12-09] @ohmann:

They are added in, but what do we want to name them? Right now they are named 'djQ Music Player'. Just as a note, this issue was branched off of issue/96 to make use of the new dropdown button.

a399610ea528 1b5fa49acf3d