Skip to content

feat: refactored tracker db folder structure. added alembic migrations #219

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
Merged
MiNeves00 merged 21 commits into from
May 8, 2025
Merged

feat: refactored tracker db folder structure. added alembic migrations #219

MiNeves00 merged 21 commits into from
May 8, 2025

Conversation

@MiNeves00
Copy link
Collaborator

@MiNeves00 MiNeves00 commented May 5, 2025 •
edited
Loading

LLMstudio Version X.X.X

What was done in this PR:

  • Large Refactor of llmstudio_tracker/db file organization into folders per type instead of per table.
  • Added alembic migrations for schema migrations
  • Added integration test for checking that writing to tracker db is working

How it was tested:

  • Integration & Unit Tests
  • Tested migrations for sqlite & postgres DB

Additional notes:

  • added alembic as a dependency

@MiNeves00 MiNeves00 requested a review from diogoncalves May 5, 2025 11:17
@MiNeves00 MiNeves00 removed the request for review from diogoncalves May 5, 2025 12:52
@MiNeves00 MiNeves00 marked this pull request as ready for review May 5, 2025 12:53
@MiNeves00 MiNeves00 marked this pull request as draft May 5, 2025 12:53
MiNeves00 added 16 commits May 5, 2025 13:55
@MiNeves00
fix: fixed the test actions for alembic
196e868
@MiNeves00
fix: fixed path for script on the test actions for alembic
630871f
@MiNeves00
fix: addded tracker dependencies to integration test action
d1817a8
@MiNeves00
fix: integration test action installation of tracker
c46ab31
@MiNeves00
fix: integration test action cache poetry of tracker
5439806
@MiNeves00
chore: updated llmstudio lib poetry lock
ef5dd14
@MiNeves00
fix: integration test action changed working dir for alembic script
1292367
@MiNeves00
fix: integration test action path for alembic.ini script
b353e6d
@MiNeves00
chore: changed the migration bash to python
890ba93
@MiNeves00
chore: changed the migration bash to python and corrected path
232ff13
@MiNeves00
fix: action
f968089
@MiNeves00
test: adding columns to check for integration tests for migrations
22ca8de
@MiNeves00
fix: alembic was not recognizing changes
c7dcfd5
@MiNeves00
fix: alembic
948c00a
@MiNeves00
chore: removed testing columns for logs
5e09f62
@MiNeves00
chore: added readme.md
174b1d7
@MiNeves00 MiNeves00 requested a review from diogoncalves May 5, 2025 16:49
@MiNeves00 MiNeves00 marked this pull request as ready for review May 5, 2025 16:49
diogoncalves
diogoncalves reviewed May 6, 2025
View reviewed changes
diogoncalves
diogoncalves approved these changes May 8, 2025
View reviewed changes
diogoncalves
diogoncalves approved these changes May 8, 2025
View reviewed changes
@MiNeves00 MiNeves00 merged commit 75b1590 into develop May 8, 2025
6 checks passed
@MiNeves00 MiNeves00 deleted the feat/add-alembic-migrations branch May 8, 2025 13:52
@MiNeves00 MiNeves00 linked an issue Jun 4, 2025 that may be closed by this pull request
FEAT: add Extras to logs_default table and do automatic migration with alembic #183
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@diogoncalves diogoncalves diogoncalves approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

FEAT: add Extras to logs_default table and do automatic migration with alembic

3 participants

@MiNeves00 @diogoncalves