Skip to content

Mac M1 Users - Link to solution #24

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

Closed
andres-zartab opened this issue Jun 8, 2021 · 4 comments
Closed

Mac M1 Users - Link to solution #24

andres-zartab opened this issue Jun 8, 2021 · 4 comments
Labels
documentation Improvements or additions to documentation

Comments

@andres-zartab
Copy link
Collaborator

andres-zartab commented Jun 8, 2021

This resource contains information on how to install TF locally for Mac M1 users: apple/tensorflow_macos#153

@andres-zartab andres-zartab added the documentation Improvements or additions to documentation label Jun 8, 2021
@mygithubid1
Copy link

It would be good to mention Mac M1 support for tensorflow in the resources section.

Here's a post from a student looking for this information:
https://community.deeplearning.ai/t/how-to-serve-the-model-using-tensorflow-serving-docker-image-on-m1-arm64-architecture/102169

@andres-zartab
Copy link
Collaborator Author

There is an additional file explaining how to get this working since Sept 2022 (this one). @cfav-dev do you think this is enough or should we also put an additional note in the resources section?

@cfav-dev
Copy link
Collaborator

cfav-dev commented Feb 3, 2023

@andres-zartab . If the additional file is on the repo and the README mentions it, I think putting a copy in the Resources is a nice-to-have but not critical. When a learner asks about the steps, we can point to the link in the README. If it's not mentioned, however, then we should add a note about it, or place it in the Resources section so mentors can refer to it. I think it's better to link to it in the notebook though. Thanks.

@andres-zartab
Copy link
Collaborator Author

In this case there is no notebook, or rather they need to go to the mkdown in order to run the notebook locally but I agree with what you said. I think it is highlighted enough so that we don't need to add this to the resources right away. Will close for now, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

3 participants