Skip to content

Conversation

amarpal
Copy link
Contributor

@amarpal amarpal commented Jul 16, 2025

Description

Updating the #Requirements section on the Installation page.

The original sentence used the word "there," which could be ambiguous about the location where Oumi cannot be installed. This change replaces "there" with "on those machines" to explicitly specify that installation is not possible on Intel Macs due to PyTorch dropping support for that platform. This improves clarity for readers and reduces potential confusion.

Related issues

Fixes # (issue)

Before submitting

  • This PR only changes documentation. (You can ignore the following checks in that case)
  • Did you read the contributor guideline Pull Request guidelines?
  • Did you link the issue(s) related to this PR in the section above?
  • Did you add / update tests where needed?

Reviewers

At least one review from a member of oumi-ai/oumi-staff is required.

The original sentence used the word "there," which could be ambiguous about the location where Oumi cannot be installed. This change replaces "there" with "on those machines" to explicitly specify that installation is not possible on Intel Macs due to PyTorch dropping support for that platform. This improves clarity for readers and reduces potential confusion.
@amarpal amarpal requested review from wizeng23, a team and taenin July 16, 2025 23:05
Copy link
Collaborator

@taenin taenin left a comment

Choose a reason for hiding this comment

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

Ship it!

@amarpal amarpal merged commit 742e743 into main Jul 16, 2025
4 checks passed
@amarpal amarpal deleted the amarpal-clarify-installation-instruction branch July 16, 2025 23:15
penfever pushed a commit that referenced this pull request Aug 27, 2025
#1831)

Updates the #Requirements section on the Installation page. The original sentence used the word "there," which could be ambiguous about the location where Oumi cannot be installed. This change replaces "there" with "on those machines" to explicitly specify that installation is not possible on Intel Macs due to PyTorch dropping support for that platform. This improves clarity for readers and reduces potential confusion.
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