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

Convert timestamp to opt-in feature #209

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rozele
Copy link
Contributor

@rozele rozele commented Oct 11, 2019

Not convinced that we need to do this, but we can spare a few bytes in the NLU results files by only including the timestamp when requested via CLI argument.

@rozele
Copy link
Contributor Author

rozele commented Oct 11, 2019

@saroup - not convinced we really need this, but we could potentially make the timestamp optional...

@rozele rozele requested review from bsherwin and removed request for bsherwin October 11, 2019 02:44
@rozele
Copy link
Contributor Author

rozele commented Nov 12, 2019

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@rozele rozele force-pushed the optInTimestamp branch 2 times, most recently from 491500b to 3893f63 Compare November 12, 2019 17:19
Not convinced that we need to do this, but we can spare a few bytes in the NLU results files by only including the timestamp when requested via CLI argument.
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.

1 participant