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.

Sign up for GitHub

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

Jump to bottom

ThejasNU/Fix init command #75

Merged
merged 4 commits into from
Feb 13, 2025
Merged

ThejasNU/Fix init command #75

merged 4 commits into from
Feb 13, 2025

Conversation

ThejasNU
Copy link
Collaborator

@ThejasNU ThejasNU commented Feb 12, 2025 •
edited
Loading

Issue addressed: #71

Tasks

  • Sentence transformer model download in init command based on environment variable
  • Raise exception from other commands if model is not downloaded
  • Update tests
  • Create query udfs while initializing the db auditor

@ThejasNU ThejasNU requested review from glennga and TJ202 and removed request for glennga February 12, 2025 10:55
@ThejasNU ThejasNU self-assigned this Feb 12, 2025
@TJ202 TJ202 merged commit aa2e6c6 into dev/0.2.0_release Feb 13, 2025
2 checks passed
@TJ202 TJ202 deleted the ThejasNU/init_cmd_fixes branch February 13, 2025 06:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TJ202 TJ202 TJ202 approved these changes

@glennga glennga Awaiting requested review from glennga

Assignees

@ThejasNU ThejasNU

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ThejasNU @TJ202