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

Allow skills to view more metadata #4

Closed
manthanhd opened this issue Oct 19, 2016 · 2 comments
Closed

Allow skills to view more metadata #4

manthanhd opened this issue Oct 19, 2016 · 2 comments

Comments

@manthanhd
Copy link
Owner

I want to see following in request object:

sentences[] // array of sentences
currentSentence  // pointer to current sentence
currentSentenceIndex  // index of the current sentence

contexts[]  // array of current and previous context states
currentContext  // pointer to current state of context
currentContextIndex  // index of the current context

topics[]  // array of topics
currentTopic  // Pointer to current topic
currentTopicIndex  // index of the current topic

Not sure how best these things should be organised.

@manthanhd
Copy link
Owner Author

Need to refine more information about the context[] storage. Do we only save the final context[] or do we save all of them? What's the limit? That could be configurable in the config with a arbitrary number being the default (20 per Id?).

@manthanhd
Copy link
Owner Author

Current version of this makes sentence and skill (replaces topic) values to be available as part of the request metadata. Closed by #9.

Can't justify recording previous context values at the moment as currently it works by reference. Will consider when another issue is raised or this is reopened.

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