Skip to content

Move all local provider things within HOME_DIR/local_provider#1575

Merged
deep1401 merged 9 commits intomainfrom
fix/s3-local-setup-status
Mar 18, 2026
Merged

Move all local provider things within HOME_DIR/local_provider#1575
deep1401 merged 9 commits intomainfrom
fix/s3-local-setup-status

Conversation

@deep1401
Copy link
Copy Markdown
Member

No description provided.

@paragon-review
Copy link
Copy Markdown

Paragon Review Skipped

Hi @deep1401! Your Polarity credit balance is insufficient to complete this review.

Please visit https://home.polarity.cc to add more credits and continue using Paragon reviews.

@sentry
Copy link
Copy Markdown

sentry bot commented Mar 18, 2026

Codecov Report

❌ Patch coverage is 20.00000% with 20 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
api/transformerlab/compute_providers/local.py 21.42% 11 Missing ⚠️
api/transformerlab/routers/compute_provider.py 22.22% 7 Missing ⚠️
..._providers/services/local/local_provider_config.py 0.00% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@deep1401 deep1401 marked this pull request as draft March 18, 2026 15:02
@deep1401 deep1401 marked this pull request as draft March 18, 2026 15:02
@deep1401 deep1401 marked this pull request as ready for review March 18, 2026 15:17
@deep1401 deep1401 changed the title Move local provider setup status to fix remote storage enabled Move all local provider things within HOME_DIR/local_provider Mar 18, 2026
"""Entry point: write server info snapshot to local_provider_config.json under HOME_DIR."""
data = asyncio.run(collect_server_info())
output_path = Path(HOME_DIR) / "local_provider_config.json"
output_path = Path(get_local_provider_root()) / "local_provider_config.json"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We call this a bunch fo times. Probably pull it into a function so we have it in a single place?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

fixed.

@deep1401 deep1401 requested a review from dadmobile March 18, 2026 18:38
@deep1401 deep1401 merged commit ea3fbe9 into main Mar 18, 2026
11 of 12 checks passed
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.

2 participants