Skip to content

weights_only=False#12

Open
stevet40 wants to merge 2 commits intojustinalsing:torch_fixfrom
stevet40:patch-1
Open

weights_only=False#12
stevet40 wants to merge 2 commits intojustinalsing:torch_fixfrom
stevet40:patch-1

Conversation

@stevet40
Copy link
Copy Markdown

@stevet40 stevet40 commented Apr 7, 2026

Trying this again, but only proposing a merge into the torch_fix branch. This fixes #10, but is maybe risky fix in the sense that it'll just unpickle anything now.

@stevet40
Copy link
Copy Markdown
Author

stevet40 commented Apr 7, 2026

Now made it a little bit safer (but defaulting to dangerous)

@stevet40
Copy link
Copy Markdown
Author

stevet40 commented Apr 7, 2026

Once this is merged I'd recommend merging torch_fix into torch, as the changes made in torch_fix considerably improve upon the current torch branch, and the Speculator model infrastructure works well on torch_fix.

@gurjeetjagwani
Copy link
Copy Markdown
Collaborator

Is this still valid? Since there is also: https://github.com/stevet40/speculator/tree/patch-2

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