Skip to content

Local vs. remote mode for auto-discovery#69

Merged
rodneykinney merged 1 commit into
mainfrom
auto-discovery
Jun 4, 2026
Merged

Local vs. remote mode for auto-discovery#69
rodneykinney merged 1 commit into
mainfrom
auto-discovery

Conversation

@rodneykinney

Copy link
Copy Markdown
Member

Add workflow for running AD both locally and on the deployed Asta service

@rodneykinney
rodneykinney requested a review from regan-huff June 3, 2026 21:27
Comment thread skills/autodiscovery/workflows/local.md Outdated

1. Ask about their research goal.
2. Inspect their datasets (`head -5 file.csv`, check JSON structure, etc.) to understand columns, units, and size.
3. Draft the configuration in whatever shape the local CLI expects (check `asta-autodiscovery --help` for the exact format — flags, config file, or stdin).

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.

does this not use the metadata.json shape expected by remote autodiscovery? would it be useful to keep input and output artifacts usable in either context?

@regan-huff regan-huff 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.

either via the hosted service (asta autodiscovery) or via the local CLI (asta-autodiscovery)
This naming and comment are confusing to me. Both are available in CLI, right? and someone running asta --help would see them both? Could we make it easier to understand which is which?

Is running locally actually free? Doesn't the user need to supply api keys for LLM usage?

@rodneykinney

rodneykinney commented Jun 4, 2026

Copy link
Copy Markdown
Member Author

Cleaned up the instructions further. Clarified need for user credentials for local mode. The local CLI actually constructs a metadata.json on the fly from the command-line parameters, so there's no need to prompt the agent to create it.

@rodneykinney
rodneykinney merged commit 40890a1 into main Jun 4, 2026
6 checks passed
@rodneykinney
rodneykinney deleted the auto-discovery branch June 4, 2026 21:34
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