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

Allow camera to have initial position and look-at vectors. #5452

Merged
merged 5 commits into from
Dec 7, 2021

Conversation

podlipensky
Copy link
Contributor

Currently in mesh plugin camera is positioned automatically which may not be convenient for applications where the model predicts both scene mesh/point cloud as well as camera position and orientation. This PR will allow to specify initial camera position and orientation as following:

const plugin_config = {
  'camera': {
    'position': [0, 0, 0],
    'look_at': [1, 1, 1]
  }
}

@stephanwlee stephanwlee merged commit 014aa80 into tensorflow:master Dec 7, 2021
yatbear pushed a commit to yatbear/tensorboard that referenced this pull request Mar 27, 2023
…w#5452)

Currently in mesh plugin camera is positioned automatically which may not be convenient for applications where the model predicts both scene mesh/point cloud as well as camera position and orientation. This PR will allow to specify initial camera position and orientation as following:

```
const plugin_config = {
  'camera': {
    'position': [0, 0, 0],
    'look_at': [1, 1, 1]
  }
}
```

Co-authored-by: Stephan Lee <[email protected]>
dna2github pushed a commit to dna2fork/tensorboard that referenced this pull request May 1, 2023
…w#5452)

Currently in mesh plugin camera is positioned automatically which may not be convenient for applications where the model predicts both scene mesh/point cloud as well as camera position and orientation. This PR will allow to specify initial camera position and orientation as following:

```
const plugin_config = {
  'camera': {
    'position': [0, 0, 0],
    'look_at': [1, 1, 1]
  }
}
```

Co-authored-by: Stephan Lee <[email protected]>
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