Skip to content
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

TensorFlow-Graphics Mesh Visualization Plugin (server side) #2168

Merged
merged 12 commits into from
May 4, 2019

Conversation

podlipensky
Copy link
Contributor

Initial pull request for tf-graphics Mesh Visualization plugin.
Note that part of client-side code of the plugin comes from tensorflow/graphics repo.
Also plugin misses demo app which will be added later.

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@googlebot
Copy link

CLAs look good, thanks!

ℹ️ Googlers: Go here for more info.

@podlipensky podlipensky changed the title TensorFlow-Graphics Mesh Visualization Plugin TensorFlow-Graphics Mesh Visualization Plugin (server side) Apr 29, 2019
Copy link
Contributor

@stephanwlee stephanwlee left a comment

Choose a reason for hiding this comment

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

Travis build is failing because you don't have the license preamble on tensorboard/plugins/mesh_visualizer/__init__.py

@stephanwlee stephanwlee requested a review from wchargin May 2, 2019 21:55
@wchargin wchargin requested review from stephanwlee and removed request for wchargin May 2, 2019 21:55
@stephanwlee stephanwlee requested review from wchargin and removed request for manivaradarajan and stephanwlee May 2, 2019 21:55
@wchargin wchargin requested review from manivaradarajan and removed request for manivaradarajan May 2, 2019 21:55
Copy link
Contributor

@wchargin wchargin left a comment

Choose a reason for hiding this comment

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

Hi! Pardon the delay; I was at a conference for the last two days.

I haven’t read this through fully, but let me get you a partial review
before I head out for the night.

tensorboard/plugins/mesh_visualizer/http_api.md Outdated Show resolved Hide resolved
tensorboard/plugins/mesh_visualizer/http_api.md Outdated Show resolved Hide resolved
tensorboard/plugins/mesh_visualizer/http_api.md Outdated Show resolved Hide resolved
tensorboard/plugins/mesh_visualizer/mesh_plugin.py Outdated Show resolved Hide resolved
tensorboard/plugins/mesh_visualizer/mesh_plugin.py Outdated Show resolved Hide resolved
tensorboard/plugins/mesh_visualizer/http_api.md Outdated Show resolved Hide resolved
tensorboard/plugins/mesh_visualizer/mesh_summary.py Outdated Show resolved Hide resolved
tensorboard/plugins/mesh_visualizer/BUILD Outdated Show resolved Hide resolved
tensorboard/plugins/mesh_visualizer/metadata.py Outdated Show resolved Hide resolved
tensorboard/plugins/mesh_visualizer/metadata.py Outdated Show resolved Hide resolved
Copy link
Contributor

@wchargin wchargin left a comment

Choose a reason for hiding this comment

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

Approving the rest to unblock you in the short term, with just a few
formatting nits in lieu of a substantive review.

tensorboard/plugins/mesh_visualizer/http_api.md Outdated Show resolved Hide resolved
tensorboard/plugins/mesh_visualizer/mesh_plugin.py Outdated Show resolved Hide resolved
tensorboard/plugins/mesh_visualizer/mesh_plugin.py Outdated Show resolved Hide resolved
tensorboard/plugins/mesh_visualizer/mesh_plugin.py Outdated Show resolved Hide resolved
tensorboard/plugins/mesh_visualizer/mesh_plugin_test.py Outdated Show resolved Hide resolved
tensorboard/plugins/mesh_visualizer/metadata_test.py Outdated Show resolved Hide resolved
tensorboard/plugins/mesh_visualizer/plugin_data.proto Outdated Show resolved Hide resolved
tensorboard/plugins/mesh_visualizer/summary_test.py Outdated Show resolved Hide resolved
tensorboard/plugins/mesh_visualizer/test_utils.py Outdated Show resolved Hide resolved
tensorboard/plugins/mesh_visualizer/test_utils.py Outdated Show resolved Hide resolved
@wchargin wchargin merged commit 22ad784 into tensorflow:master May 4, 2019
stephanwlee added a commit to stephanwlee/tensorboard that referenced this pull request May 6, 2019
At PR tensorflow#2168, it was recommended to call mesh_visualizer plugin a mesh
plugin. As a result, the Python code lives in tensorboard/plugins/mesh
while the frontend lives in tensorboard/plugins/mesh_visualizer. This
change moves frontend code into t/p/mesh.
stephanwlee added a commit that referenced this pull request May 21, 2019
At PR #2168, it was recommended to call mesh_visualizer plugin a mesh
plugin. As a result, the Python code lives in tensorboard/plugins/mesh
while the frontend lives in tensorboard/plugins/mesh_visualizer. This
change moves frontend code into t/p/mesh.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants