PLTF-2905: Expand Laminar observability and config guidance for OHE#579
Conversation
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
|
@juanmichelini Just a quick run to see if the agent sees something. Please feel free to not consider it if it has its moment of exaggerations 😅 @OpenHands Take a look at this PR and verify Laminar docs and clone yourself agent-sdk and openhands repos from the same org if you need. Review it. Post your feedback as a gh review with gh api. |
|
I'm on it! enyst can track my progress at all-hands.dev |
enyst
left a comment
There was a problem hiding this comment.
Reviewed the docs and cross-checked the Laminar env vars against OpenHands/software-agent-sdk (LMNR_PROJECT_API_KEY, LMNR_BASE_URL, LMNR_FORCE_HTTP) and OpenHands env forwarding (AUTO_FORWARD_PREFIXES). The Laminar doc URLs resolve and git diff --check passes. I left one inline concern about the Enterprise endpoint example using a concrete all-hands.dev host in a guide that otherwise uses <your-base-domain>.
This review was created by an AI agent (OpenHands) on behalf of the user.
This comment was marked as duplicate.
This comment was marked as duplicate.
Updated base URLs in configuration examples to use placeholders for user-defined domains.
Co-authored-by: openhands <openhands@all-hands.dev>
rajshah4
left a comment
There was a problem hiding this comment.
Great work pull this all together, looks good to me
Summary
LMNR_/LLM_variables, and post-setup Laminar workflowsTesting
This PR description was created by an AI agent (OpenHands) on behalf of the user.