Skip to content

feat: Onboard New York Trees dataset#265

Merged
adlersantos merged 17 commits into
GoogleCloudPlatform:mainfrom
gkodukula:new_york_trees
May 19, 2022
Merged

feat: Onboard New York Trees dataset#265
adlersantos merged 17 commits into
GoogleCloudPlatform:mainfrom
gkodukula:new_york_trees

Conversation

@gkodukula

@gkodukula gkodukula commented Jan 10, 2022

Copy link
Copy Markdown
Contributor

Description

Pipelines included:

  • new_york_trees.tree_census_2005
  • new_york_trees.tree_census_2015

Note: It's recommended to open an issue first for context and discussion.

Checklist

Note: Delete items below that aren't applicable to your pull request.

  • Please merge this PR for me once it is approved.
  • If this PR adds or edits a dataset or pipeline, it was reviewed and approved by the Google Cloud Public Datasets team beforehand.
  • If this PR adds or edits a dataset or pipeline, I put all my code inside datasets/new_york_trees and nothing outside of that directory.
  • This PR is appropriately labeled.

@nlarge-google nlarge-google left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some small changes

Comment thread datasets/new_york_trees/pipelines/_images/run_csv_transform_kub/csv_transform.py Outdated
Comment thread datasets/new_york_trees/pipelines/_images/run_csv_transform_kub/csv_transform.py Outdated
Comment thread datasets/new_york_trees/pipelines/tree_census_2005/pipeline.yaml
Comment thread datasets/new_york_trees/pipelines/tree_census_2005/pipeline.yaml
Comment thread datasets/new_york_trees/pipelines/tree_census_2015/pipeline.yaml Outdated
Comment thread datasets/new_york_trees/pipelines/tree_census_2015/tree_census_2015_dag.py Outdated
Comment thread datasets/new_york_trees/pipelines/tree_census_2015/tree_census_2015_dag.py Outdated
@nlarge-google

Copy link
Copy Markdown
Contributor

@adlersantos - I have reviewed - please move to prod. Thanks!

@adlersantos adlersantos merged commit 2905308 into GoogleCloudPlatform:main May 19, 2022
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.

3 participants