Skip to content

Conversation

@crisely09
Copy link

I had a few issues installing mlcroissant in a recently formatted MacBook Pro.
This PR adds the steps I had to take to solve those issues to the README file.

@crisely09 crisely09 requested a review from a team as a code owner February 6, 2025 14:23
@github-actions
Copy link

github-actions bot commented Feb 6, 2025

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

@ccl-core
Copy link
Contributor

ccl-core commented Feb 7, 2025

recheck

@ccl-core
Copy link
Contributor

ccl-core commented Feb 7, 2025

Hi @crisely09 , thank you for your contribution! :) Can you please sign the MLCommons CLA?

@crisely09
Copy link
Author

Hi @crisely09 , thank you for your contribution! :) Can you please sign the MLCommons CLA?

Hi, I have sent the CLA already by email. Thank you!

@crisely09
Copy link
Author

crisely09 commented Feb 7, 2025

I am member of MLCommons organization, the CLA check should pass now, I think.

pygraphviz
```

The command can fail also due to missing a rust compiler. The standar way to install it is:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
The command can fail also due to missing a rust compiler. The standar way to install it is:
The command can fail also due to missing a rust compiler. The standard way to install it is:

### macOS possible issues

In the case of macOS graphviz and pygraphviz can be installed with brew, running:
```
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
```
```shell

```

The command can fail also due to missing a rust compiler. The standar way to install it is:
```
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
```
```shell

Copy link
Contributor

@bact bact left a comment

Choose a reason for hiding this comment

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

Fix a small typo and add "shell" language code to two code blocks.

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.

4 participants