New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Value Tracer #886

Closed

Giuseppe5 opened this issue Feb 29, 2024 · 0 comments · Fixed by #888

Collaborator

Giuseppe5 commented Feb 29, 2024

The last changes introduced in https://github.com/Xilinx/brevitas/blame/200456825f3b4b8db414f2b25b64311f82d3991a/src/brevitas/fx/value_tracer.py#L231

Should be undone and value_tracer compatibility with certain models (e.g., Llama2) should be tested.

Giuseppe5 mentioned this issue

Fix (proxy): revert value tracer change #888

Merged

Giuseppe5 linked a pull request

that will close this issue

Fix (proxy): revert value tracer change #888

Merged

Giuseppe5 closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment