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

does autoEdit2 use the X-Watson-Learning-Opt-Out true header for Watson transcriptions? #43

Open
pietrop opened this issue Jan 11, 2018 · 0 comments

Comments

@pietrop
Copy link
Collaborator

pietrop commented Jan 11, 2018

As originally raised by @Riptwo on twitter.

does autoEdit2 use the X-Watson-Learning-Opt-Out true header for Watson transcriptions?

Shotr answer:

No.

Long answer:

It uses the watson-developer-cloud node module. But I haven't specified that option (I think maybe it wasn't around when I first did it)

I see now it has been added as an option in the SDK node module

At the moment as mentioned there, by default all request are logged

Possible fix:

To implement, it be a matter of adding

var myInstance = new watson.WhateverServiceV1({
  /* username, password, version, etc... */
  headers: {
    "X-Watson-Learning-Opt-Out": true
  }
});

Where the Watson STT instance is initialised changing it as follows to add the headers informations.

  speech_to_text = watson.speech_to_text({
	    username: keys.username,
	    password: keys.password,
	    version: 'v1',
 	    headers: {
                 "X-Watson-Learning-Opt-Out": true
              }
	  });

Things to think about for integration (ie going beyond hard coding a solution):

In the UI/UX It would also be good to add this as an option when creating the transcription, or in the settings where the credentials are added to give more fine grained control to the user on whether to opt in or out, rather then hard coding it.

Help and feedback wanted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant