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

modifications to run caching notebook #44

Merged
merged 1 commit into from
Mar 6, 2025
Merged

modifications to run caching notebook #44

merged 1 commit into from
Mar 6, 2025

Conversation

haesleinhuepf
Copy link
Member

@haesleinhuepf haesleinhuepf commented Mar 1, 2025

Hi @lea-33 ,

here comes the caching notebook after executing it. I had to do some small modifications to make it work. Please check them carefully. Also note: If you rename your environment variable HUGGINGFACE_TOKEN to HF_TOKEN, you do not have to login manually anymore. HF will do it then automatically.

Also note, I didn't have write-access to your repository no HF and thus created this one: https://huggingface.co/datasets/haesleinhuepf/SlightInsight_Cache

Running the notebook caused costs of $6.69 btw.

closes #42

Best,
Robert

@lea-33
Copy link
Collaborator

lea-33 commented Mar 6, 2025

Hi @haesleinhuepf ,

looks good, thank you!
I will now take care of the copyright issue you mentioned in #45 ! Thanks for reminding me of that!

@lea-33 lea-33 merged commit 8f5350c into main Mar 6, 2025
1 check passed
@haesleinhuepf
Copy link
Member Author

Good question @lea-33 ! I cannot find the option to give you write-access. The settings say you can send pull-requests. To be honest, I'm also quite new to huggingface...

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.

Calculate Embeddings for all entries
2 participants