Skip to content
This repository was archived by the owner on Nov 30, 2023. It is now read-only.

Conversation

@bhack
Copy link
Contributor

@bhack bhack commented Jun 23, 2020

Add Tensorflow devcontainer

@bhack bhack mentioned this pull request Jun 23, 2020
@Chuxel
Copy link
Member

Chuxel commented Jun 24, 2020

@bhack Let me know when you think this is to the point it should be reviewed. I assume you're still making edits like you had for the tenserflow_addons PR.

@bhack
Copy link
Contributor Author

bhack commented Jun 24, 2020

No this is totally independent cause it is not waiting for a upstream image. We are wrapping on the official one. So if you want you can make a pass. I suppose that pylint is enabled by default right?

@Chuxel Chuxel self-requested a review June 24, 2020 14:15
Copy link
Member

@Chuxel Chuxel left a comment

Choose a reason for hiding this comment

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

Looks good - one suggestion below. LMK if you want me to merge or have anyone else review.

@bhack
Copy link
Contributor Author

bhack commented Jun 24, 2020

@Chuxel If you can try it on GPU we could merge.

@Chuxel
Copy link
Member

Chuxel commented Jun 24, 2020

Unfortunately, I don't have a Linux machine with a NVIDIA GPU in it. (My Linux laptop has an Intel GPU.) My Azure VMs aren't using GPU SKUs at the moment either unfortunately.

@bhack
Copy link
Contributor Author

bhack commented Jun 24, 2020

Ok let's merge we will see if somebody with GPU will open a ticket.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants