Skip to content

Add input text to logs #42

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

Open
1 task done
Ataraxist opened this issue Apr 18, 2025 · 1 comment · May be fixed by #46
Open
1 task done

Add input text to logs #42

Ataraxist opened this issue Apr 18, 2025 · 1 comment · May be fixed by #46
Assignees
Labels
enhancement New feature or request

Comments

@Ataraxist
Copy link
Contributor

Ataraxist commented Apr 18, 2025

Problem

The logs are currently detached from the inputs, so while there is a UUID setup for associating the inputs to the the similarity log entry, there is no log of inputs anywhere. I imagine that as data analysts use this tool, it would be important to see the specific inputs that are causing drift on either side of the scale, user and AI.

Solution

We can solve this problem by adding the input raw text to some sort of additional file, like a JSONL or even just JSON file that has the UUID and input text, this would make the log entry for any specific drift violation have a UUID as a pointer to the JSONL that contains the input text.

Another solution: The function could be made to accommodate a UUID as a third parameter so that, in a production environment, any number of I/Os can be linked together in the log.

Additional information

No response

👨‍👧‍👦 Contributing

  • 🙋‍♂️ Yes, I'd love to make a PR to implement this feature!
@Ataraxist Ataraxist added the enhancement New feature or request label Apr 18, 2025
@Ataraxist
Copy link
Contributor Author

This is in progress, pending a PR approval for the merge of feat/logInputText into dev, and then subsequently into Master.

NPM package has been incremented to 1.1.0 from 1.0.7 to reflect this feature.

@Ataraxist Ataraxist self-assigned this Apr 27, 2025
@Ataraxist Ataraxist linked a pull request Apr 27, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant