Skip to content
This repository has been archived by the owner on Aug 28, 2021. It is now read-only.

Target $I$ for the Gated-LSTM baseline #63

Open
lionminhu opened this issue Jan 14, 2020 · 4 comments
Open

Target $I$ for the Gated-LSTM baseline #63

lionminhu opened this issue Jan 14, 2020 · 4 comments

Comments

@lionminhu
Copy link

Hello, I have been trying to implement the Gated-LSTM baseline according to the descriptions in the RoomNav paper (link), but I am confused with the some of the notations.

From Section 5.2.2 (for Gated-LSTM):
image

The symbol $I$ is used to denote the target instruction. This same symbol was used in Section 5.1.2 (for Gated-CNN) to denote the target instruction prior to conversion to an embedding vector $y$:
image

So if we were to follow the same notation as in Section 5.1.2, we would have to feed the one-hot vector $I$ (and not the embedding vector) as input to the gated fusion module and the LSTM for the Gated-LSTM.

However, the Figure 2 seems to imply that it is in fact the embedding vector that is fed into the gated fusion and the LSTM:
image

Could you clarify whether the target $I$ for Gated-LSTM baseline is meant to be the one-hot vector or the embedding vector?

Thank you in advance.

@GoingMyWay
Copy link

@lionminhu Hi, where did you get the SUNCG data to install this library?

@lionminhu
Copy link
Author

Previously (as in minosworld/minos#94), we were able to request the authors for the SUNCG dataset.
Due to legal issues (citation), however, the download links and the forms have all been removed (c727331).
I do not think the dataset is going to be available again in a while, if at all.

Relevant: #57, minosworld/minos#159

@GoingMyWay
Copy link

Previously (as in minosworld/minos#94), we were able to request the authors for the SUNCG dataset.
Due to legal issues (citation), however, the download links and the forms have all been removed (c727331).
I do not think the dataset is going to be available again in a while, if at all.

Relevant: #57, minosworld/minos#159

Yes, can you share the link for downloading the data or do you know how to use other datasets to fix this issue?

@lionminhu
Copy link
Author

Unfortunately, I am not aware of any download links that work anymore.
My knowledge is possibly outdated, but if house dataset/environment is what you need, you may also want to consider GibsonEnv with Matterport3D dataset.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants