Skip to content

Conversation

@johnnygreco
Copy link
Contributor

@johnnygreco johnnygreco commented Dec 9, 2025

This PR adds some initial plugin documentation. Note that we expect this area to evolve a lot over the next couple months.

I also added analysis to the code reference.

Closes #107

@johnnygreco johnnygreco force-pushed the johnny/docs/76-add-plugin-documentation branch from 6caa6e3 to 2b637b0 Compare December 10, 2025 19:54
Comment on lines 131 to 137
completion_tokens_mean: Union[float, MissingValue]
completion_tokens_median: Union[float, MissingValue]
completion_tokens_stddev: Union[float, MissingValue]
prompt_tokens_mean: Union[float, MissingValue]
prompt_tokens_median: Union[float, MissingValue]
prompt_tokens_stddev: Union[float, MissingValue]
column_type: Literal[DataDesignerColumnType.LLM_TEXT.value] = DataDesignerColumnType.LLM_TEXT.value
Copy link
Contributor

Choose a reason for hiding this comment

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

Just noticed the use of completion/prompt tokens here. In this PR: I'm updating this to input/output tokens. If this PR merges first, I'll update these.

Copy link
Contributor

Choose a reason for hiding this comment

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

made the update in 51dcffa

Copy link
Contributor

@nabinchha nabinchha left a comment

Choose a reason for hiding this comment

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

Need one small update!

@johnnygreco johnnygreco force-pushed the johnny/docs/76-add-plugin-documentation branch from 3ca576b to 378acb4 Compare December 11, 2025 20:33
@johnnygreco
Copy link
Contributor Author

@nabinchha – addressed your comments in 378acb4

Copy link
Contributor

@nabinchha nabinchha left a comment

Choose a reason for hiding this comment

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

:shipit:

@johnnygreco johnnygreco merged commit 48fdc8c into main Dec 11, 2025
13 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.

3 participants