Skip to content

Conversation

@nuvic
Copy link
Contributor

@nuvic nuvic commented May 5, 2025

🗣 Description

Without GITHUB_TOKEN, it hits the rate limit right away:

WARN runtime::init::dataset: The dataset spiceai.issues (github) has been rate limited. GitHub API rate limit exceeded. Try again later, and add a LIMIT clause to your query to reduce the number of requests.

🤔 Concerns

If a new user reading this doesn't have their GITHUB_TOKEN in their .env, it increases the friction to trying out this core feature of spice.ai. An alternative to this example is to use a csv file hosted on github:

  - from: https://raw.githubusercontent.com/LearnDataSci/articles/refs/heads/master/Python%20Pandas%20Tutorial%20A%20Complete%20Introduction%20for%20Beginners/IMDB-Movie-Data.csv
    name: movie_reviews
    acceleration:
      enabled: true
    params:
      file_format: csv
    columns:
      - name: Description
        embeddings:
          - from: remote_service

nuvic added 2 commits May 5, 2025 15:46
Without `gh_token`, it hits the rate limit right away.
@lukekim lukekim enabled auto-merge (squash) May 7, 2025 03:51
@lukekim
Copy link
Contributor

lukekim commented May 7, 2025

Thank you!

@lukekim lukekim disabled auto-merge May 7, 2025 04:09
@lukekim lukekim enabled auto-merge (squash) May 7, 2025 04:09
@nuvic
Copy link
Contributor Author

nuvic commented May 8, 2025

Hello, is there anything else I need to do for it to be merged?

@lukekim lukekim merged commit 8cf930a into spiceai:trunk May 12, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants