Skip to content

testing(data-science-onramp): add debug output - #5343

Merged
gcf-merge-on-green[bot] merged 4 commits into
GoogleCloudPlatform:masterfrom
tmatsuo:data-science-onramp
Feb 9, 2021
Merged

testing(data-science-onramp): add debug output#5343
gcf-merge-on-green[bot] merged 4 commits into
GoogleCloudPlatform:masterfrom
tmatsuo:data-science-onramp

Conversation

@tmatsuo

@tmatsuotmatsuo commented Feb 8, 2021

Copy link
Copy Markdown
Contributor

fixes#5312

It won't fix the issue, but will allow us to debug better.

Takashi Matsuo
testing(data-science-onramp): add debug output
633de8f
fixesGoogleCloudPlatform#5312
It won't fix the issue, but will allow us to debug better.
@tmatsuo
tmatsuo requested review from a team, bradmiro and leahecole as code ownersFebruary 8, 2021 19:33
@product-auto-labelproduct-auto-labelBot added the samples Issues that are directly related to samples. label Feb 8, 2021
@google-clagoogle-claBot added the cla: yes This human has signed the Contributor License Agreement. label Feb 8, 2021
@tmatsuo

tmatsuo commented Feb 8, 2021

Copy link
Copy Markdown
ContributorAuthor

The debug output for the success case:

df: tripduration starttime ... gender customer_plan 0 778 s 2018-05-31T23:59:59.606 ... MALE NaN 1 12.45 min 2018-05-31T23:59:59.386 ... m NaN 2 0.13805555555555554 h 2018-05-31T23:59:51.083 ... MALE NaN 3 NaN 2018-05-31T23:59:48.001 ... MALE NaN 4 0.13416666666666666 h 2018-05-31T23:59:45.677 ... F NaN .. ... ... ... ... ... 95 511 s 2018-05-31T23:54:28.590 ... m NaN 96 780 s 2018-05-31T23:54:23.967 ... m NaN 97 16.016666666666666 min 2018-05-31T23:54:19.876 ... U NaN 98 824 s 2018-05-31T23:54:08.100 ... f NaN 99 195 s 2018-05-31T23:54:07.112 ... male NaN [100 rows x 16 columns] job_config: <google.cloud.bigquery.job.load.LoadJobConfig object at 0x7f971b4affd0>

@tmatsuotmatsuo mentioned this pull request Feb 9, 2021
Closed
@busunkim96busunkim96 added the automerge Merge the pull request once unit tests and other checks pass. label Feb 9, 2021
@gcf-merge-on-green
gcf-merge-on-greenBot merged commit 1078b51 into GoogleCloudPlatform:masterFeb 9, 2021
@gcf-merge-on-greengcf-merge-on-greenBot removed the automerge Merge the pull request once unit tests and other checks pass. label Feb 9, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bradmirobradmirobradmiro approved these changes
@leahecoleleahecoleAwaiting requested review from leahecole
+1 more reviewer
@busunkim96busunkim96busunkim96 approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@busunkim96busunkim96

Labels

cla: yesThis human has signed the Contributor License Agreement.samplesIssues that are directly related to samples.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

data-science-onramp.data-cleaning.clean_test: test_clean failed

3 participants

@tmatsuo@busunkim96@bradmiro