search

dhruv-suraj / Project

1 stars 0 forks source link

improvements needed #1

Open Disha-Patil opened 2 years ago

Disha-Patil commented 2 years ago
  1. clean the data, there are some values in text e.g. 'nil' which are showing in the numeric columns. Please look at yaxis of your first two plots. Nil should be replaced with a number.
  2. when you are plotting the line graph for years, do not plot total, average, and maximum on same plot. This distorts the information we can read from the line graph.
  3. For density plot, why are the damage values in negative ? also put legends on the density plot.
  4. Please include a map chart, just like the US data one.
dhruv-suraj commented 2 years ago

I will definitely do the required changes and get back to you.

On Tue, Jun 28, 2022 at 9:51 AM Disha-Patil @.***> wrote:

  1. clean the data, there are some values in text e.g. 'nil' which are showing in the numeric columns. Please look at yaxis of your first two plots. Nil should be replaced with a number.
  2. when you are plotting the line graph for years, do not plot total, average, and maximum on same plot. This distorts the information we can read from the line graph.
  3. For density plot, why are the damage values in negative ? also put legends on the density plot.
  4. Please include a map chart, just like the US data one.

— Reply to this email directly, view it on GitHub https://github.com/dhruv-suraj/Project/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASQSPFSKAS6TA2UWMQYT4CLVRJ4TRANCNFSM52AW2E6A . You are receiving this because you are subscribed to this thread.Message ID: @.***>

-- Disclaimer This email is governed by the Disclaimer Terms of  somaiya.edu http://somaiya.edu which may be viewed at  https://www.somaiya.edu/en/email-disclaimer https://www.somaiya.edu/en/email-disclaimer

Finally, please do not print this email unless it is necessary. Every unprinted email helps the environment.

dhruv-suraj commented 2 years ago

I have worked upon the required changes, but haven't been able to form a map layout because there are no latitude and longitude is there any other way I can form the map.

On Tue, Jun 28, 2022 at 10:15 AM DHRUV SURAJ @.***> wrote:

I will definitely do the required changes and get back to you.

On Tue, Jun 28, 2022 at 9:51 AM Disha-Patil @.***> wrote:

  1. clean the data, there are some values in text e.g. 'nil' which are showing in the numeric columns. Please look at yaxis of your first two plots. Nil should be replaced with a number.
  2. when you are plotting the line graph for years, do not plot total, average, and maximum on same plot. This distorts the information we can read from the line graph.
  3. For density plot, why are the damage values in negative ? also put legends on the density plot.
  4. Please include a map chart, just like the US data one.

— Reply to this email directly, view it on GitHub https://github.com/dhruv-suraj/Project/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASQSPFSKAS6TA2UWMQYT4CLVRJ4TRANCNFSM52AW2E6A . You are receiving this because you are subscribed to this thread.Message ID: @.***>

-- Disclaimer This email is governed by the Disclaimer Terms of  somaiya.edu http://somaiya.edu which may be viewed at  https://www.somaiya.edu/en/email-disclaimer https://www.somaiya.edu/en/email-disclaimer

Finally, please do not print this email unless it is necessary. Every unprinted email helps the environment.