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

Add local experiment visualisation to urbanair cli #659

Draft
wants to merge 39 commits into
base: master
Choose a base branch
from

Conversation

defaultobject
Copy link
Collaborator

@defaultobject defaultobject commented Feb 28, 2021

Summary

Type of change

  • Bug fix
  • New feature
  • Infrastructure
  • API
  • Pipeline

Additional dependencies

  • geopandas==0.8.2
  • rtree==0.9.7

Tests

How has the code been tested?

Issues fixes

List any issues this closes:

Checklist

Please ensure you have done the following:

  • Code conforms to the projects style guidelines

  • New code documented using google docstyle

  • Updated documentation of any changed code

  • Any additional dependencies have been added to the appropriate location (e.g. setup.py or requirements.txt)

  • Unit tests written and passing on local database. Tests should prove that fix is effective or new feature works as expected

  • Code needs to run on the cluster? Tag database administrator in the pull request review to configure.

Reviewer

@defaultobject defaultobject added the New feature Adds a new feature label Feb 28, 2021
@defaultobject defaultobject self-assigned this Feb 28, 2021
@PatrickOHara PatrickOHara changed the base branch from master to add_inducing_point_experiments March 9, 2021 15:13
Base automatically changed from add_inducing_point_experiments to master March 16, 2021 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New feature Adds a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant