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

How to change Client GRPC deadline? #283

Closed
bloodcarter opened this issue Dec 22, 2016 · 6 comments
Closed

How to change Client GRPC deadline? #283

bloodcarter opened this issue Dec 22, 2016 · 6 comments
Labels
🚨 This issue needs some love. triage me I really want to be triaged.

Comments

@bloodcarter
Copy link

I randomly receive this error while streaming data:
Client GRPC deadline too short. Should be at least: 3 * audio-duration + 5 seconds. Current deadline is: 59.699356980447192 second(s). Required at least: 60 second(s).

What should I do with that?

@jmdobry
Copy link
Member

jmdobry commented Dec 22, 2016

@jmuk How would one increase the deadline?

@jmuk
Copy link

jmuk commented Dec 27, 2016

The autogen layer can customize the deadline through the CallOption parameter, however @google-cloud/speech hand-written layer doesn't allow passing timeout by themselves.

Which method are you using? I'm wild-guessing that asyncRecognize does not have such restriction.

@jmdobry
Copy link
Member

jmdobry commented Feb 8, 2017

Something like this would work:

const v1beta1 = require('@google-cloud/speech').v1beta1();
const speech = v1beta1.speechClient();

const request = {
  audio: {
    uri: 'gs://my-bucket/filename.raw'
  },
  config: {
    encoding: 'LINEAR16',
    sampleRate: 16000
  }
};

const options = {
  longrunning: {
    initialRetryDelayMillis: 100, , // Default
    retryDelayMultiplier: 1.3, // Default
    maxRetryDelayMillis: 60000, // Default
    initialRpcTimeoutMillis: null, // Default
    rpcTimeoutMultiplier: null, // Default
    maxRpcTimeoutMillis: null, // Default
    totalTimeoutMillis: 600000 // Default
  }
};

speech.syncRecognize(request, options)
  .then((results) => {
    const transcription = results[0];

    console.log(transcription);
  });

@jmdobry jmdobry closed this as completed Feb 8, 2017
@bloodcarter
Copy link
Author

@jmdobry Would this work with recognizeStream?

@jmdobry
Copy link
Member

jmdobry commented Feb 9, 2017

I'm not sure the bi-directional streaming uses exponential backoff. @jmuk?

@jmuk
Copy link

jmuk commented Feb 9, 2017

@bloodcarter: createRecognizeStream can customize the timeout as https://github.com/GoogleCloudPlatform/google-cloud-node/blob/master/packages/speech/src/index.js#L378

const speech = require('@google-cloud')();
const stream = speech.createRecognizeStream(..., {timeout: 300});
...

@jmdobry: bidi streaming does not use exponential backoff, but uses timeout field in the options.

const v1beta1 = require('@google-cloud/speech').v1beta1();
const speech = v1beta1.speechClient();

// set the timeout to 300 seconds.
const stream = speech.streamingRecognize({timeout: 300});
stream.write(...);
...

This should customize the timeout.

@yoshi-automation yoshi-automation added 🚨 This issue needs some love. triage me I really want to be triaged. labels Apr 6, 2020
NimJay pushed a commit that referenced this issue Nov 10, 2022
* updated CHANGELOG.md [ci skip]

* updated package.json [ci skip]

* updated samples/package.json [ci skip]
NimJay pushed a commit that referenced this issue Nov 10, 2022
* updated CHANGELOG.md [ci skip]

* updated package.json [ci skip]

* updated samples/package.json [ci skip]
ace-n pushed a commit that referenced this issue Nov 11, 2022
ace-n pushed a commit that referenced this issue Nov 11, 2022
…ssing samples (#283)

* docs(samples): add OCR, form, quality, splitter and specialized processing samples

* 🦉 Updates from OwlBot

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

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
ace-n pushed a commit that referenced this issue Nov 11, 2022
ace-n pushed a commit that referenced this issue Nov 11, 2022
…ssing samples (#283)

* docs(samples): add OCR, form, quality, splitter and specialized processing samples

* 🦉 Updates from OwlBot

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

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
ace-n pushed a commit that referenced this issue Nov 12, 2022
…ssing samples (#283)

* docs(samples): add OCR, form, quality, splitter and specialized processing samples

* 🦉 Updates from OwlBot

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

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
ace-n pushed a commit that referenced this issue Nov 14, 2022
fix(deps): update dependency nodemailer to v6

This PR contains the following updates:

| Package | Type | Update | Change | References |
|---|---|---|---|---|
| nodemailer | dependencies | major | [`^5.0.0` -> `^6.0.0`](https://diff.intrinsic.com/nodemailer/5.1.1/6.0.0) | [homepage](https://nodemailer.com/), [source](https://github.com/nodemailer/nodemailer) |

---

### Release Notes

<details>
<summary>nodemailer/nodemailer</summary>

### [`v6.0.0`](https://github.com/nodemailer/nodemailer/blob/master/CHANGELOG.md#&#8203;600-2019-03-25)

[Compare Source](https://github.com/nodemailer/nodemailer/compare/v5.1.1...v6.0.0)

-   SMTPConnection: use removeListener instead of removeAllListeners (xr0master) [ddc4af15]
    Using removeListener should fix memory leak with Node.js streams

</details>

---

### Renovate configuration

:date: **Schedule**: "after 9am and before 3pm" (UTC).

:vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

:recycle: **Rebasing**: Whenever PR is stale, or if you modify the PR title to begin with "`rebase!`".

:no_bell: **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- renovate-rebase -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/marketplace/renovate). View repository job log [here](https://renovatebot.com/dashboard#googleapis/nodejs-compute).

#283 automerged by dpebot
ace-n pushed a commit that referenced this issue Nov 14, 2022
ace-n pushed a commit that referenced this issue Nov 14, 2022
…ssing samples (#283)

* docs(samples): add OCR, form, quality, splitter and specialized processing samples

* 🦉 Updates from OwlBot

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

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
ace-n pushed a commit that referenced this issue Nov 15, 2022
ace-n pushed a commit that referenced this issue Nov 15, 2022
ace-n pushed a commit that referenced this issue Nov 17, 2022
ace-n pushed a commit that referenced this issue Nov 17, 2022
ahrarmonsur pushed a commit that referenced this issue Nov 17, 2022
ahrarmonsur pushed a commit that referenced this issue Nov 18, 2022
🤖 I have created a release \*beep\* \*boop\*
---
## [3.9.0](https://github.com/googleapis/nodejs-secret-manager/compare/v3.8.1...v3.9.0) (2021-07-30)


### Features

* In Secret Manager, users can now use filter to customize the output of ListSecrets/ListSecretVersions calls ([#283](https://github.com/googleapis/nodejs-secret-manager/issues/283)) ([063691a](https://github.com/googleapis/nodejs-secret-manager/commit/063691a87291fab82037e9b444812dcf0724eb0c))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
ace-n pushed a commit that referenced this issue Nov 21, 2022
fix(deps): update dependency nodemailer to v6

This PR contains the following updates:

| Package | Type | Update | Change | References |
|---|---|---|---|---|
| nodemailer | dependencies | major | [`^5.0.0` -> `^6.0.0`](https://diff.intrinsic.com/nodemailer/5.1.1/6.0.0) | [homepage](https://nodemailer.com/), [source](https://github.com/nodemailer/nodemailer) |

---

### Release Notes

<details>
<summary>nodemailer/nodemailer</summary>

### [`v6.0.0`](https://github.com/nodemailer/nodemailer/blob/master/CHANGELOG.md#&#8203;600-2019-03-25)

[Compare Source](https://github.com/nodemailer/nodemailer/compare/v5.1.1...v6.0.0)

-   SMTPConnection: use removeListener instead of removeAllListeners (xr0master) [ddc4af15]
    Using removeListener should fix memory leak with Node.js streams

</details>

---

### Renovate configuration

:date: **Schedule**: "after 9am and before 3pm" (UTC).

:vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

:recycle: **Rebasing**: Whenever PR is stale, or if you modify the PR title to begin with "`rebase!`".

:no_bell: **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- renovate-rebase -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/marketplace/renovate). View repository job log [here](https://renovatebot.com/dashboard#googleapis/nodejs-compute).

#283 automerged by dpebot
ace-n pushed a commit that referenced this issue Nov 21, 2022
fix(deps): update dependency nodemailer to v6

This PR contains the following updates:

| Package | Type | Update | Change | References |
|---|---|---|---|---|
| nodemailer | dependencies | major | [`^5.0.0` -> `^6.0.0`](https://diff.intrinsic.com/nodemailer/5.1.1/6.0.0) | [homepage](https://nodemailer.com/), [source](https://github.com/nodemailer/nodemailer) |

---

### Release Notes

<details>
<summary>nodemailer/nodemailer</summary>

### [`v6.0.0`](https://github.com/nodemailer/nodemailer/blob/master/CHANGELOG.md#&#8203;600-2019-03-25)

[Compare Source](https://github.com/nodemailer/nodemailer/compare/v5.1.1...v6.0.0)

-   SMTPConnection: use removeListener instead of removeAllListeners (xr0master) [ddc4af15]
    Using removeListener should fix memory leak with Node.js streams

</details>

---

### Renovate configuration

:date: **Schedule**: "after 9am and before 3pm" (UTC).

:vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

:recycle: **Rebasing**: Whenever PR is stale, or if you modify the PR title to begin with "`rebase!`".

:no_bell: **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- renovate-rebase -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/marketplace/renovate). View repository job log [here](https://renovatebot.com/dashboard#googleapis/nodejs-compute).

#283 automerged by dpebot
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚨 This issue needs some love. triage me I really want to be triaged.
Projects
None yet
Development

No branches or pull requests

4 participants