-
Notifications
You must be signed in to change notification settings - Fork 30
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
FastMLX Python Client #23
Comments
Q: Why do you need a standalone client? Couldn't you set all of these variables by API? |
Yes, you can set the variables. But this would help if you want to programmatically start and stop the server. Imagine like the OpenAI/Anthrophic Python Client |
I ask out of complete ignorance, but would one of the following approaches from ChatGPT work? Using Using
|
Feature Description
Implement a FastMLX client that allows users to specify custom server settings, including base URL, port, and number of workers. This feature will provide greater flexibility for users who want to run the FastMLX server with specific configurations.
Proposed Implementation
Modify the
FastMLX
class constructor to accept additional parameters:base_url
: str (default: "http://localhost:8000")workers
: int (default: 2)Update the
FastMLXClient
class to:base_url
to extract host and portworkers
parameterModify the
start_fastmlx_server
function to accepthost
,port
, andworkers
as parameters.Update the
ensure_server_running
method inFastMLXClient
to use the custom settings when starting the server.Example Usage
Benefits
Potential Challenges
Tasks
FastMLX
class constructorFastMLXClient
to handle custom settingsstart_fastmlx_server
functionensure_server_running
methodQuestions
Please provide any feedback or suggestions on this proposed implementation.
The text was updated successfully, but these errors were encountered: