Skip to content

Conversation

@shreyashkgupta
Copy link
Collaborator

No description provided.

@shreyashkgupta shreyashkgupta linked an issue Mar 19, 2025 that may be closed by this pull request
@shreyashkgupta
Copy link
Collaborator Author

@sauravpanda please review this once

// SmolLM2-135M-Instruct
/\/([\w\.-]+)\-(\d+\.?\d*[BM])\-Instruct/,
// SmolLM2-135M
/\/([\w\.-]+)\-(\d+\.?\d*[BM])/
Copy link
Owner

Choose a reason for hiding this comment

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

I think this pattern may not have all the model names.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Okay, let me recheck

clean: true,
sourcemap: true,
treeshake: true,
treeshake: false,
Copy link
Owner

Choose a reason for hiding this comment

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

what does this do?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

If we set treeshake as true, and the function we write is not used within code, it skips exporting it. The clearModelCache function needs it

@sauravpanda sauravpanda merged commit 020cd6d into main Mar 19, 2025
4 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.

Add auto cache clearing

3 participants