Skip to content

Conversation

@husseinmozannar
Copy link
Contributor

Minor fixes to enable windows support for fara-cli and install instructions.

Closes #32

Copy link
Contributor Author

Choose a reason for hiding this comment

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

formatting only

Copy link
Contributor Author

Choose a reason for hiding this comment

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

formatting only

all_observations.append(action_description)
self.logger.info(f"Observation#{i+1}: {action_description}")
self.logger.debug(f"Observation#{i+1}: {action_description}")
print(f"Observation#{i+1}: {action_description}")
Copy link
Contributor Author

Choose a reason for hiding this comment

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

windows dont show the logger so added print

Issues = "https://github.com/microsoft/fara/issues"

[project.optional-dependencies]
vllm = ["vllm>=0.10.0"]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

vllm crashes windows install, so kept optional


Notes:
- If you're using Windows, we highly recommend using WSL2 (Windows Subsystem for Linux).
- If you're using Windows, we highly recommend using WSL2 (Windows Subsystem for Linux). Please the Windows instructions in the [Installation](#Installation) section.

Choose a reason for hiding this comment

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

Please the Windows instructions in the Installation section

Missing word?
Please FOLLOW the Windows instructions...?
Please VIEW the Windows instructions...?

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.

Fix Windows support for vllm model serving or alternative

3 participants