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

adding implementation of https://arxiv.org/abs/1805.06066 #5265

Merged
merged 2 commits into from
Sep 11, 2018

Conversation

arsalan-mousavian
Copy link
Contributor

No description provided.

@arsalan-mousavian arsalan-mousavian requested a review from a team as a code owner September 7, 2018 16:21
@robieta
Copy link
Contributor

robieta commented Sep 7, 2018

Thanks @arsalan-mousavian. Typically, models in the Research repository require a collaborator at Google Research to take ownership. Are you working with someone there, or do you have somebody who is willing to act as reviewer and owner of the code?

@arsalan-mousavian
Copy link
Contributor Author

This is the work I did last year in Brain Robotics. I think @ayzaan @mfiser or @Toshev can review this PR. I am not sure if they are on the brain-github team as well.

@robieta
Copy link
Contributor

robieta commented Sep 7, 2018

Lovely. Thanks for letting me know.

### TODO: SUNCG Dataset
Current version of the code does not support SUNCG dataset. It can be added by
implementing necessary functions of `envs/task_env.py` using the public
releaseed code of SUNCG environment such as
Copy link

Choose a reason for hiding this comment

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

released

## 3. Training
Right now, the released version only supports training and inference using the real data from Active Vision Dataset.

When RGB image modality is used Resnet the embedding is initialized is used To start the training download pre-trained Resnet50 check point in the working directory ./resnet_v2_50_checkpoint/resnet_v2_50.ckpt
Copy link

Choose a reason for hiding this comment

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

seems like a typo here.
Maybe something like:
When the RGB image modality is used, the Resnet embeddings are initialized.

@arsalan-mousavian
Copy link
Contributor Author

Addressed the comments. Please take a look.

Copy link
Contributor

@robieta robieta left a comment

Choose a reason for hiding this comment

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

Excellent. For my own reference, approval link:

go/cognitive_planning_oss_approval

@robieta robieta merged commit e6ce8cd into tensorflow:master Sep 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants