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

Feature/basic-chat-app-with-flagging #1

Merged
merged 9 commits into from
Dec 2, 2024

Conversation

burtenshaw
Copy link
Collaborator

This PR includes a basic chatting app that stores feedback in a dataframe.

Next step: push dataframe to dataset repo

@burtenshaw
Copy link
Collaborator Author

@burtenshaw https://huggingface.co/docs/huggingface_hub/main/en/guides/upload#scheduled-uploads

Nice. Are you suggesting that I use it for the upload step?

@davidberenstein1957
Copy link
Member

@burtenshaw It might help for inspiration. I am not sure how I implemented the flagging myself, but I believe it logs continuously (every interaction) instead of periodically as the implementation above but I guess it could work either way.

@burtenshaw
Copy link
Collaborator Author

@burtenshaw It might help for inspiration. I am not sure how I implemented the flagging myself, but I believe it logs continuously (every interaction) instead of periodically as the implementation above but I guess it could work either way.

I'll add it to give us options. But we're still early so need to decide.

@burtenshaw
Copy link
Collaborator Author

@davidberenstein1957 I've added persistence, so now I think we should move on to inference. The only outstanding issue are unique identifiers for user and session, this will make it easier for us to deal with the data downstream.

@davidberenstein1957
Copy link
Member

@burtenshaw I will have a look later today.

@davidberenstein1957 davidberenstein1957 merged commit 72a56d8 into main Dec 2, 2024
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.

2 participants