Blog #271
Blog #271
Conversation
![ellipsis-dev[bot]](https://avatars.githubusercontent.com/in/64358?s=60&v=4){kind=small}
There was a problem hiding this comment.
👍 Looks good to me! Reviewed everything up to 7507df2 in 19 seconds
More details
- Looked at
215lines of code in4files - Skipped
3files when reviewing. - Skipped posting
1drafted comments based on config settings.
1. frontend/app/blog/[slug]/page.tsx:51
- Draft comment:
Thelevelforh1is set to 0, which is invalid. It should start from 1. Adjust the level accordingly. - Reason this comment was not posted:
Comment was not on a valid diff hunk.
Workflow ID: wflow_zpmnrRoED8biuodM
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.
There was a problem hiding this comment.
👍 Looks good to me! Incremental review on cf71d84 in 8 seconds
More details
- Looked at
13lines of code in1files - Skipped
1files when reviewing. - Skipped posting
1drafted comments based on config settings.
1. frontend/package.json:80
- Draft comment:
Ensure that the update toposthog-jsversion^1.194.4is compatible with the rest of the codebase and does not introduce breaking changes. - Reason this comment was not posted:
Confidence changes required:50%
The change in the version ofposthog-jsshould be verified for compatibility and potential breaking changes.
Workflow ID: wflow_Hmi0MVMJC3eL14yL
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.
|
|
||
| ### 1. Span Paths | ||
|
|
||
| Span paths uniquely identify where LLM calls happen in your code. They're automatically constructed from the location of the call, making it easy to track specific functions and endpoints. |
There was a problem hiding this comment.
nit: maybe a doc link to https://docs.lmnr.ai/tracing/structure#grouping-spans-into-traces
But not necessary, those docs don't specialize on path
| {data.image && | ||
| <div className="w-full flex items-center py-4"> | ||
| <Image src={data.image} alt={data.title} width={1200} height={800} /> | ||
| <div className="w-full flex rounded overflow-hidden"> |
There was a problem hiding this comment.
while we're at it, do we need border here?
Important
Update blog layout, add new content for online evaluations, and adjust
BlogMetacomponent layout.page.tsxto center content and adjust margins.text-white/85andtext-whiterespectively.page.tsxandblog-meta.tsxtomd:w-[1000px].2024-12-06-online-evals.mdxdetailing online evaluations feature.2024-12-01-launch-week-1.mdxto include additional launch days and descriptions.BlogMetacomponent inblog-meta.tsxto adjust layout and image handling.posthog-jsversion inpackage.json.This description was created by
for cf71d84. It will automatically update as commits are pushed.