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

Adds geographically weighted regression analyses (describe and predict) #250

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

andy-esch
Copy link
Contributor

@andy-esch andy-esch commented Jan 17, 2017

For new analysis use the following checklist:

  • Uses the_geom geometry(Geometry, 4326) column for all geospatial operations.
  • Outputs a the_geom geometry(Geometry, 4326) column.
  • Outputs a cartodb_id numeric column.
  • Uses {cache: true} option when it needs full knowledge of the table it. Hints: aggregations, window functions.
  • Uses {cache: true} if it access external services.
  • Naming uses a-z lowercase and hyphens.
  • All mandatory params cannot be made optional.
  • Avoids using CTEs for join operations when result is not cached.

@jgoizueta
Copy link
Contributor

Before merging/releasing this, please check with me @andy-esch, we have released a version of camshaft that needs some fixing before being used

@andy-esch
Copy link
Contributor Author

Thanks @jgoizueta, I'll take a look 👍

@andy-esch
Copy link
Contributor Author

We might pause on this for a while until we know it will get the Builder Analysis interface created as well. Until then we're going to create dep sql node for this instead.

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

Successfully merging this pull request may close these issues.

2 participants