-
Notifications
You must be signed in to change notification settings - Fork 607
Rename module etdb to inspector #908
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.
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
Conversation
✅ Deploy Preview for resplendent-gnome-14e531 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
This pull request was exported from Phabricator. Differential Revision: D50282527 |
bc19f1d
to
e6b7104
Compare
This pull request was exported from Phabricator. Differential Revision: D50282527 |
1 similar comment
This pull request was exported from Phabricator. Differential Revision: D50282527 |
e6b7104
to
92dfe1d
Compare
This pull request was exported from Phabricator. Differential Revision: D50282527 |
92dfe1d
to
c7a31f8
Compare
This pull request was exported from Phabricator. Differential Revision: D50282527 |
c7a31f8
to
f3175f7
Compare
This pull request was exported from Phabricator. Differential Revision: D50282527 |
f3175f7
to
cfb743b
Compare
This pull request was exported from Phabricator. Differential Revision: D50282527 |
Summary: Etdb directory was renamed, but because the Example usage is static, it didn't get updated Reviewed By: Jack-Khuu Differential Revision: D50282527
cfb743b
to
d0d32ed
Compare
This pull request was exported from Phabricator. Differential Revision: D50282527 |
This pull request has been merged in d71efe4. |
Summary: Pull Request resolved: #908 Etdb directory was renamed, but because the Example usage is static, it didn't get updated Reviewed By: tarun292, Jack-Khuu Differential Revision: D50282527 fbshipit-source-id: 4839fcc74dc9c5905bfacfe30f4f199968c9541f
Summary: Pull Request resolved: #908 Etdb directory was renamed, but because the Example usage is static, it didn't get updated Reviewed By: tarun292, Jack-Khuu Differential Revision: D50282527 fbshipit-source-id: 4839fcc74dc9c5905bfacfe30f4f199968c9541f
Summary: Pull Request resolved: #908 Etdb directory was renamed, but because the Example usage is static, it didn't get updated Reviewed By: tarun292, Jack-Khuu Differential Revision: D50282527 fbshipit-source-id: 4839fcc74dc9c5905bfacfe30f4f199968c9541f
Introduces basic Python dataclasses to represent requests, responses, and associated objects defined in the OpenAI API specification. This will enable creating a basic server that follows the spec so users can leverage TorchChat to run LLMs on their own hardware with a familiar interface to existing cloud tools. **Testing** Lintunner ``` lintrunner Warning: Could not find a lintrunner config at: '.lintrunner.private.toml'. Continuing without using configuration file. >>> Lint for generate.py: Advice (FLAKE8) C901 'Generator.chat' is too complex (32) See https://www.flake8rules.com/rules/C901.html. 581 | buffer.clear() 582 | # print(, end='', flush=True) 583 | >>> 584 | def chat( 585 | self, 586 | generator_args: GeneratorArgs, 587 | ): ``` (advice from prior commit) Dataclasses are used and tested further in PR #908
Remove the existing browser UI and replace it with a UI built with Streamlit. This reduces complexity & leverages the functionality introduced in PR #906 to display chunked responses. **Testing** ``` streamlit run torchchat.py -- browser stories110M --compile --max-new-tokens 256 You can now view your Streamlit app in your browser. Local URL: http://localhost:8501 Network URL: http://192.0.0.2:8501 ``` <img width="1002" alt="image" src="https://github.com/user-attachments/assets/df305943-2326-4d01-a48b-61dd2006fa28">
Summary: Etdb directory was renamed, but because the Example usage is static, it didn't get updated
Differential Revision: D50282527