Skip to content

Conversation

elasticdotventures
Copy link

No description provided.

d6e
d6e previously approved these changes May 30, 2025
Copy link
Owner

@d6e d6e left a comment

Choose a reason for hiding this comment

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

lgtm

@d6e d6e dismissed their stale review May 30, 2025 22:57

I realized I overlooked several issues

@elasticdotventures
Copy link
Author

No worries. Are you going to merge it? (i'll switch back to your version rather than running on my fork)

Copy link
Owner

@d6e d6e left a comment

Choose a reason for hiding this comment

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

Okay yeah, actually I think it's fine

elasticdotventures and others added 9 commits July 5, 2025 05:45
Output is truncated to fit the specified token limit using Hugging Face's pretrained bert-base-cased tokenizer.
🦨 Skunky: Truncation is by character, not true token boundary; see code comment for future improvement.
All changes build cleanly and are isolated to the explicit requirements.
version 0.2.0; --tldr --max_tokens & new list_crate_items (uses rust analyzer)
@elasticdotventures
Copy link
Author

FYI - not sure why this wasn't merged, but I've continued to add features to my fork.

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