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

Linear Relation Visualization feature request #24

Open
apoorvagnihotri opened this issue May 13, 2020 · 1 comment
Open

Linear Relation Visualization feature request #24

apoorvagnihotri opened this issue May 13, 2020 · 1 comment

Comments

@apoorvagnihotri
Copy link
Collaborator

apoorvagnihotri commented May 13, 2020

linearRelation

I think adding the following features would help better understand the plot.

  • What are the blue dots representing? A legend would help with this.
  • Xlabel should instead be the actual years.
  • The left ylabel could instead be Pollutant ratio (SO2/NOX).
    If that is what is meant.

@prerna-khanna Hi Prerna, can you please take this up?

apoorvagnihotri added a commit that referenced this issue May 14, 2020
apoorvagnihotri added a commit that referenced this issue May 14, 2020
@apoorvagnihotri
Copy link
Collaborator Author

apoorvagnihotri commented May 14, 2020

Currently, I am not sure about the correctness of the code here. The new linearRelation generates the following plot.

img

If the line is just the average of the then why should we have 2 y-ticks?

In addition to that, the lines that I have highlighted below also need to be checked from openAir and addressed accordingly.

pol_1 = df_new[pol1].mean()
# this might not be entirely correct.
# TODO: we want to take the average of the slope.
# [Not take the average and find the slope]
# Additionally, we would like to have the option of
# changing the granularity.
pol_2 = df_new[pol2].mean()
values.append(pol_2 / pol_1)

Additionally, do we fit a linear relation of just take the ratio of the two pollutants? I think we need to fit a linear relation and then find the slope.

Original LinearRelation - https://github.com/davidcarslaw/openair/blob/master/R/linearRelation.R

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant