docs: add DetectIntent samples by aribray · Pull Request #500 · googleapis/java-dialogflow-cx

aribray · 2022-06-29T16:01:08Z

Adds the following DetectIntent samples:

detectIntent with IntentInput
detectIntent with EventInput
detectIntent with synthesized TTS output
detectIntent with disabled webhook
detectIntent with sentiment analysis
streamingDetectIntent with partial response enabled
detectIntent with AudioInput

…red sample and test Change-Id: I75bed489bd8abefca60263acb266501810c0e856

Change-Id: I18be2a5c1d46ac9838cf4a15ab87dcf331ee26ee

Change-Id: Ia85899f678ef1382718c2443a682770417340af2

Change-Id: I4dd56107a3197027f287980fa87a15e962154783

Change-Id: I8dcdb0dcaad486a2fb69e0737df99bda49b28839

Change-Id: I5f4521b3e50660f30f9f73da3d939da213084d28

Change-Id: I39dedfba1b62be5cb4580a42dde270146f3b6f97

Change-Id: Ic54ae0d1f091079661f3fa67097cdcd2e888dc90

Change-Id: Ifd303b7395844240dca0b29cde9b791112fcabe5

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Change-Id: I59e45da9f9e57b327a527f1405816ac1275fe1c8

Change-Id: Ic2eb71f5d30ea0c53f9fe2264aa4b2a3f5cec3c8

Change-Id: I70ff4118cc8b3111a3ed55a2fae32258d3f6819c

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Change-Id: I62b7e34fef632635b3281d60058ee40000b211ef

…alogflow-cx into aribray--webhook-samples Change-Id: I751bab7fd1a6d098da1475a82eb63785fe30464a

Change-Id: I395dd5abc9a48733126ab0ecdc2a974fbf9eb471

Change-Id: Ia599ab7f4d432477d9952b4e12c709845f1ad97b

Change-Id: I8cdf93fd9e21161757d198b1338c5ab77bb3a236

Change-Id: I90f1f626588fd59c116d119348054c914945cd1d

Change-Id: I5ff686ab2ffc94b358bdf87b8dfc3d00f5e45657

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Change-Id: I4a2bef88da7ed04d343da63544563130182e6aaf

snippet-bot · 2022-06-29T16:01:25Z

Here is the summary of changes.

You are about to add 7 region tags.

samples/snippets/src/main/java/dialogflow/cx/DetectIntentAudioInput.java:19, tag dialogflow_cx_v3_detect_intent_audio_input
samples/snippets/src/main/java/dialogflow/cx/DetectIntentDisableWebhook.java:19, tag dialogflow_cx_v3_detect_intent_disable_webhook
samples/snippets/src/main/java/dialogflow/cx/DetectIntentEventInput.java:19, tag dialogflow_cx_v3_detect_intent_event_input
samples/snippets/src/main/java/dialogflow/cx/DetectIntentIntentInput.java:19, tag dialogflow_cx_v3_detect_intent_intent_input
samples/snippets/src/main/java/dialogflow/cx/DetectIntentSentimentAnalysis.java:19, tag dialogflow_cx_v3_detect_intent_sentiment_analysis
samples/snippets/src/main/java/dialogflow/cx/DetectIntentStreamingPartialResponse.java:19, tag dialogflow_cx_v3_detect_intent_streaming_partial_response
samples/snippets/src/main/java/dialogflow/cx/DetectIntentSynthesizeTextToSpeechOutput.java:19, tag dialogflow_cx_v3_detect_intent_synthesize_tts_output

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

Refresh this comment

Change-Id: I8e6a26441a79b1ed44def4877c822040cb143e33

…ava-dialogflow-cx into aribray--detect-intent-samples Change-Id: I665bbab77d49baa7626f101da5e2e1442b176f37

Change-Id: I741c73d3f14bfe5f9b4ff6ec55d43f2687ac51dd

…ava-dialogflow-cx into aribray--detect-intent-samples Change-Id: I82b8512833a44c451f7f754022a67ddb7e50fbbd

lesv

A few suggestions / questions:

where UUID's are expected, perhaps you should actually create UUID's.
Method comments may be appropriate in places.
Tests don't appear to understand that they can be running in 9 separate instances at once.
I don't see a lot of cleaning up of resources in tests. Both what your creating in the moment, and stuff that might be left over after a crash.

samples/snippets/src/main/java/dialogflow/cx/DetectIntentAudioInput.java

samples/snippets/src/main/java/dialogflow/cx/DetectIntentDisableWebhook.java

samples/snippets/src/main/java/dialogflow/cx/DetectIntentEventInput.java

lesv · 2022-07-06T23:22:05Z

samples/snippets/src/test/java/dialogflow/cx/DetectIntentIntentInputTest.java

+  private static String AGENT_ID =
+      System.getenv()
+          .getOrDefault("DIALOGFLOW_CX_AGENT_ID_GLOBAL", "b8d0e85d-0741-4e6d-a66a-3671184b7b93");
+  private static String INTENT_ID = "00000000-0000-0000-0000-000000000000";


Should these be real UUID's? Remember you might have 9 of these running at the same time in different processes / processors.

Each agent has a unique Agent ID that's created by Dialogflow. There was a permanent "TestAgent" resource created for the samples, and it has its own Flow IDs, Page IDs, etc. that are also created by Dialogflow (00000000-0000-0000-0000-000000000000 is the Default Welcome Intent Flow ID).

Am I understanding the question correctly? I can go in and create a new intent by hand as well, if that makes more sense.

lesv · 2022-07-06T23:22:37Z

samples/snippets/src/test/java/dialogflow/cx/DetectIntentSentimentAnalysisTest.java

+  private static String AGENT_ID =
+      System.getenv()
+          .getOrDefault("DIALOGFLOW_CX_AGENT_ID_GLOBAL", "b8d0e85d-0741-4e6d-a66a-3671184b7b93");


Change-Id: Iac48159892b0b6a34f31b013d446dfd66e62b8a7

Change-Id: I8e3fcea6a24739cb7a1e49d7572688e22d4d0147

Change-Id: I29f4cb587b10f46576aa36a56cde8c8fcb6e1acb

Change-Id: I327b7e90d8562c9698bc1fc4324ec249c6326436

Change-Id: If9937d41cefcff2fa0480524d036aecf5a1fda40

Change-Id: I9980a07e9c2ca377f669f0b80bbf16a42033d634

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

…t-samples Change-Id: I22b72222ea9567ff525dfcbc08aac4753e9ef1ce

Change-Id: I5115e749e47427c50e2ed0ad494525dc6f9be66a

Change-Id: Ibe5abeee5b6b003fc8c77eb53eb432a340629255

aribray and others added 23 commits May 18, 2022 17:39

docs:add configure webhook to set form parameter as optional or requi…

27217aa

…red sample and test Change-Id: I75bed489bd8abefca60263acb266501810c0e856

docs: fix checkstyle

9ec52bf

Change-Id: I18be2a5c1d46ac9838cf4a15ab87dcf331ee26ee

fix checkstyle

47d91ad

Change-Id: Ia85899f678ef1382718c2443a682770417340af2

fix checkstyle

fec80c6

Change-Id: I4dd56107a3197027f287980fa87a15e962154783

fix multiline string

7826893

Change-Id: I8dcdb0dcaad486a2fb69e0737df99bda49b28839

fix checkstyle violations

b0ea884

Change-Id: I5f4521b3e50660f30f9f73da3d939da213084d28

fix checkstyle violations

acd6659

Change-Id: I39dedfba1b62be5cb4580a42dde270146f3b6f97

remove unused variable

d9fb522

Change-Id: Ic54ae0d1f091079661f3fa67097cdcd2e888dc90

fix checkstyle violations

fedcc6b

Change-Id: Ifd303b7395844240dca0b29cde9b791112fcabe5

🦉 Updates from OwlBot post-processor

3b3d65f

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

fix checkstyle violations

aec26af

Change-Id: I59e45da9f9e57b327a527f1405816ac1275fe1c8

fix checkstyle violations

984a4bd

Change-Id: Ic2eb71f5d30ea0c53f9fe2264aa4b2a3f5cec3c8

fix checkstyle violations

4428653

Change-Id: I70ff4118cc8b3111a3ed55a2fae32258d3f6819c

🦉 Updates from OwlBot post-processor

f6d9d1d

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Merge remote-tracking branch 'origin/main' into aribray--webhook-samples

006eda1

Change-Id: I62b7e34fef632635b3281d60058ee40000b211ef

Merge branch 'aribray--webhook-samples' of github.com:aribray/java-di…

7c35bf0

…alogflow-cx into aribray--webhook-samples Change-Id: I751bab7fd1a6d098da1475a82eb63785fe30464a

remove unnecessary samples

fe9b212

Change-Id: I395dd5abc9a48733126ab0ecdc2a974fbf9eb471

add webhook configure session parameter sample and test

77ba085

Change-Id: Ia599ab7f4d432477d9952b4e12c709845f1ad97b

add webhook validate form parameter sample and test

16ab5b8

Change-Id: I8cdf93fd9e21161757d198b1338c5ab77bb3a236

fix checkstyle violations

7bf63a0

Change-Id: I90f1f626588fd59c116d119348054c914945cd1d

fix checkstyle violations

5176e43

Change-Id: I5ff686ab2ffc94b358bdf87b8dfc3d00f5e45657

🦉 Updates from OwlBot post-processor

9d65836

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

add detectintent with intent input sample and test

c6600ab

Change-Id: I4a2bef88da7ed04d343da63544563130182e6aaf

product-auto-label bot added size: l Pull request size is large. api: dialogflow Issues related to the googleapis/java-dialogflow-cx API. labels Jun 29, 2022

product-auto-label bot added the samples Issues that are directly related to samples. label Jun 29, 2022

aribray added 3 commits June 29, 2022 11:02

docs: add detect intent samples and tests

20a4af4

Change-Id: I8e6a26441a79b1ed44def4877c822040cb143e33

Merge branch 'aribray--detect-intent-samples' of github.com:aribray/j…

7bc2f49

…ava-dialogflow-cx into aribray--detect-intent-samples Change-Id: I665bbab77d49baa7626f101da5e2e1442b176f37

add detectintent with audio input sample and test

7d4dad9

Change-Id: I741c73d3f14bfe5f9b4ff6ec55d43f2687ac51dd

Merge branch 'aribray--detect-intent-samples' of github.com:aribray/j…

4cce0dd

…ava-dialogflow-cx into aribray--detect-intent-samples Change-Id: I82b8512833a44c451f7f754022a67ddb7e50fbbd

aribray added the owlbot:run Add this label to trigger the Owlbot post processor. label Jun 30, 2022

gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jun 30, 2022

aribray added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 1, 2022

yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 1, 2022

kweinmeister approved these changes Jul 1, 2022

View reviewed changes

lesv approved these changes Jul 6, 2022

View reviewed changes

aribray added 6 commits July 7, 2022 08:20

hardcode parameters

73ab951

Change-Id: Iac48159892b0b6a34f31b013d446dfd66e62b8a7

move sessionID comments

7f03607

Change-Id: I8e3fcea6a24739cb7a1e49d7572688e22d4d0147

add main and hardcode variables

a9da7a4

Change-Id: I29f4cb587b10f46576aa36a56cde8c8fcb6e1acb

add session path comments and path format

613028c

Change-Id: I327b7e90d8562c9698bc1fc4324ec249c6326436

checkstyle

5bb7eda

Change-Id: If9937d41cefcff2fa0480524d036aecf5a1fda40

checkstyle

cd8d6c6

Change-Id: I9980a07e9c2ca377f669f0b80bbf16a42033d634

aribray added the owlbot:run Add this label to trigger the Owlbot post processor. label Jul 7, 2022

gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jul 7, 2022

gcf-owl-bot bot and others added 3 commits July 7, 2022 16:48

🦉 Updates from OwlBot post-processor

1eeeec7

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Merge branch 'main' into aribray--detect-intent-samples

2d8a47e

Merge remote-tracking branch 'origin/main' into aribray--detect-inten…

f146dce

…t-samples Change-Id: I22b72222ea9567ff525dfcbc08aac4753e9ef1ce

aribray added the owlbot:run Add this label to trigger the Owlbot post processor. label Jul 12, 2022

gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jul 12, 2022

aribray added the owlbot:run Add this label to trigger the Owlbot post processor. label Jul 12, 2022

gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jul 12, 2022

aribray added 2 commits July 12, 2022 07:17

fix list construction

d4569f8

Change-Id: I5115e749e47427c50e2ed0ad494525dc6f9be66a

import ArrayList

b68e81c

Change-Id: Ibe5abeee5b6b003fc8c77eb53eb432a340629255

aribray added the owlbot:run Add this label to trigger the Owlbot post processor. label Jul 12, 2022

gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jul 12, 2022

aribray merged commit 2c5f9c2 into googleapis:main Jul 12, 2022

aribray deleted the aribray--detect-intent-samples branch July 12, 2022 12:50

release-please bot mentioned this pull request Jul 12, 2022

chore(main): release 0.14.1 #512

Merged

blunderbuss-gcf bot assigned lesv Jul 20, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

docs: add DetectIntent samples #500

docs: add DetectIntent samples #500
aribray merged 61 commits intogoogleapis:mainfrom
aribray:aribray--detect-intent-samples

aribray commented Jun 29, 2022 •

edited

Loading

Uh oh!

snippet-bot bot commented Jun 29, 2022 •

edited

Loading

Uh oh!

lesv left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

lesv Jul 6, 2022

Uh oh!

aribray Jul 7, 2022

Uh oh!

lesv Jul 6, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

aribray commented Jun 29, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

snippet-bot bot commented Jun 29, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

lesv left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

lesv Jul 6, 2022

Choose a reason for hiding this comment

Uh oh!

aribray Jul 7, 2022

Choose a reason for hiding this comment

Uh oh!

lesv Jul 6, 2022

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

aribray commented Jun 29, 2022 •

edited

Loading

snippet-bot bot commented Jun 29, 2022 •

edited

Loading