Skip to content
This repository was archived by the owner on Dec 11, 2025. It is now read-only.

chore: Add tqdm to project deps#92

Merged
DLesage25 merged 1 commit into
mainfrom
d3-add-tqdm-dependency
Aug 12, 2025
Merged

chore: Add tqdm to project deps#92
DLesage25 merged 1 commit into
mainfrom
d3-add-tqdm-dependency

Conversation

@DLesage25

Copy link
Copy Markdown
Contributor

tqdm is used on the SDK (e.g. toolkit/rag/evaluation.py) but not listed on the project dependencies. This cases certain installations to raise a missing dependency error.

@r0ymanesco r0ymanesco left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests are failing most likely because we updated the staging supabase setup so not sure if the keys still exist. You can sign in to the staging frontend using https://staging-app.notdiamond.ai/login

You have to make sure to sign up using email/pwd because the google auth will only work on the prod instance

@github-actions

github-actions Bot commented Aug 12, 2025

Copy link
Copy Markdown

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  notdiamond/toolkit/rag
  evaluation_dataset.py
Project Total  

This report was generated by python-coverage-comment-action

@DLesage25 DLesage25 merged commit 1f5de8f into main Aug 12, 2025
3 of 6 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants