kaustubhhiware / rose

Analyse all kinds of data for a TV series
https://rose-tv.herokuapp.com
MIT License
37 stars 13 forks source link

(Webapp) : Minimum rating specified, interactive plots #5

Open kaustubhhiware opened 5 years ago

kaustubhhiware commented 5 years ago

Once the webapp is completed, and a plot is visualised for any show, we should have a bar / input field that takes an imdb rating for instance, and highlights the episodes with rating higher than the specified value.

Could proceed with displaying the plot first, and then highlighting the bars where value >= specified. Visualization to be discussed before sending PR.

Pointed out by Shalini Mukhopadhyay

shaliniiit commented 5 years ago

I am working on this please.

kaustubhhiware commented 5 years ago

@shaliniiit Thanks for your enthusiasm, however this issue can be worked upon only after #4 is completed, since this is a dependent task. I'm aware you're the one who pointed this out, but as you can see this task depends on a previous issue which is currently being worked upon.

shaliniiit commented 5 years ago

Respected sir As of now even I have not been able to completely grasp the code and get it running on Anaconda. So I will be very happy to work once #7 gets over as I need time myself. thanks. Regards.

[image: Mailtrack] https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality5& Sender notified by Mailtrack https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality5& 12/12/18, 10:48:51

On Wed, 12 Dec 2018 at 10:37, Kaustubh Hiware notifications@github.com wrote:

@shaliniiit https://github.com/shaliniiit Thanks for your enthusiasm, however this issue can be worked upon only after #4 https://github.com/kaustubhhiware/rose/issues/4 is completed, since this is a dependent task. I'm aware you're the one who pointed this out, but as you can see this task depends on a previous issue which is currently being worked upon.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/kaustubhhiware/rose/issues/5#issuecomment-446463805, or mute the thread https://github.com/notifications/unsubscribe-auth/AfCiinV8tkxkG1y0hjMy6-I_q_R3S67Kks5u4I8vgaJpZM4ZHzde .

kaustubhhiware commented 5 years ago

In that case, assigning this issue to you. Will notify you once the previous issue is completed.

shaliniiit commented 5 years ago

Respected sir I would be really obliged if you could let me know if the previous issue has been resolved. Thanks

On Wed, 12 Dec 2018 at 2:01 PM, Kaustubh Hiware notifications@github.com wrote:

In that case, assigning this issue to you. Will notify you once the previous issue is completed.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/kaustubhhiware/rose/issues/5#issuecomment-446504097, or mute the thread https://github.com/notifications/unsubscribe-auth/AfCiir4dzwGRLLJDPZtL7xwlmx9Fly_qks5u4L76gaJpZM4ZHzde .

kaustubhhiware commented 5 years ago

Yes, so the issue is now currently available since the PR for #4 was merged only today.

We need to do the following : Have an interactive plot instead of the currently static one. For this, we will modify the scrape_views.py script such that for command line usage, it uses Matplotlib and for webapp, it generates a plotly or d3 plot, which is interactive.

Have a look at #26. Currently a variable named decider is used to distinguish usage between script and webapp calling it, and an image is generated which is hardly the recommended approach.

kaustubhhiware commented 5 years ago

@shaliniiit if I was not able to make myself clear, or you have trouble understanding anything, feel free to comment here / mail me. I have intentionally not given a detailed solution straight away, but mentioned only keywords so that you can learn most of the stuff on your own rather than me just handing it out to you.

shaliniiit commented 5 years ago

Now if the decider==2 we are using show(). What I must do is generate a d3 plot instead of just printing a 2d image?

shaliniiit commented 5 years ago

I am getting Internal Server Error The server encountered an internal error and was unable to complete your request. Either the server is overloaded or there is an error in the application. whenever I am trying to display Bar chart and Average plot for a TV series without making any changes to the original code. Is there some special tool I have to install?

kaustubhhiware commented 5 years ago

When exactly are you getting the error? Seems to be working fine for me.

shaliniiit commented 5 years ago

When I hit submit for a TV series it gives this error

On Tue, 25 Dec 2018 at 7:47 PM, Kaustubh Hiware notifications@github.com wrote:

When exactly are you getting the error? Seems to be working fine for me.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/kaustubhhiware/rose/issues/5#issuecomment-449853701, or mute the thread https://github.com/notifications/unsubscribe-auth/AfCiilp7RuUkfXSyslJdkUub7vvlsGjaks5u8jNhgaJpZM4ZHzde .

shaliniiit commented 5 years ago

For display bar chart and display averaged chart

On Tue, 25 Dec 2018 at 7:48 PM, Shalini Mukhopadhyay < mukhopadhyayshalini2@gmail.com> wrote:

When I hit submit for a TV series it gives this error

On Tue, 25 Dec 2018 at 7:47 PM, Kaustubh Hiware notifications@github.com wrote:

When exactly are you getting the error? Seems to be working fine for me.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/kaustubhhiware/rose/issues/5#issuecomment-449853701, or mute the thread https://github.com/notifications/unsubscribe-auth/AfCiilp7RuUkfXSyslJdkUub7vvlsGjaks5u8jNhgaJpZM4ZHzde .

kaustubhhiware commented 5 years ago

Please tick imdb option, and then tick one of averaged or bar chart. Should work.

Since you're developing, instead of running the webapp you should try to execute the flask webapp on your own system using the instructions specified in the README.

shaliniiit commented 5 years ago

I have ticked both options and tried to run it on prompt as well on which it gives Module not found for scrape_views. When I try to install scrape_views it says version not found. I am trying on another laptop now. Thanks.

On Tue, 25 Dec 2018 at 11:02 PM, Kaustubh Hiware notifications@github.com wrote:

Please tick imdb option, and then tick one of averaged or bar chart. Should work.

Since you're developing, instead of running the webapp you should try to execute the flask webapp on your own system using the instructions specified in the README.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/kaustubhhiware/rose/issues/5#issuecomment-449864632, or mute the thread https://github.com/notifications/unsubscribe-auth/AfCiinGVJiTUI9ysnWos5JgJhURCKT4Vks5u8mE0gaJpZM4ZHzde .

shaliniiit commented 5 years ago

It is running now. But now there is a connectivity problem in this area. [image: image.png]

[image: image.png] .How much time do I have? I think it will be alright in half an hour. Thanks.

[image: Mailtrack] https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality5& Sender notified by Mailtrack https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality5& 26/12/18, 01:06:15

On Tue, 25 Dec 2018 at 23:38, Shalini Mukhopadhyay < mukhopadhyayshalini2@gmail.com> wrote:

I have ticked both options and tried to run it on prompt as well on which it gives Module not found for scrape_views. When I try to install scrape_views it says version not found. I am trying on another laptop now. Thanks.

On Tue, 25 Dec 2018 at 11:02 PM, Kaustubh Hiware notifications@github.com wrote:

Please tick imdb option, and then tick one of averaged or bar chart. Should work.

Since you're developing, instead of running the webapp you should try to execute the flask webapp on your own system using the instructions specified in the README.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/kaustubhhiware/rose/issues/5#issuecomment-449864632, or mute the thread https://github.com/notifications/unsubscribe-auth/AfCiinGVJiTUI9ysnWos5JgJhURCKT4Vks5u8mE0gaJpZM4ZHzde .

shaliniiit commented 5 years ago

In my case the app is showing bar chart and average plot correctly only for Game of Thrones and not for Two and a Half Men. And running scrape_views.py is giving this error image

shaliniiit commented 5 years ago

image Can you help me with this error please..