We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
I really appreciate your work and thanks a lot for sharing this work with us!
I am now following your code step by step to understand the model and apply it to my data. And while I'm doing this, I got a question.
In the line 187-192 of data_loader.py The code is written like this.
187 def getitem(self, index): *** 190 return np.float32(self.train[index:index + self.win_size]), np.float32(self.test_labels[0:self.win_size]) 192 return np.float32(self.val[index:index + self.win_size]), np.float32(self.test_labels[0:self.win_size])
Shouldn't the '0' be changed to 'index'? self.test_labels[0:self.win_size] -> self.test_labels[index:index + self.win_size]
If there's anything that I missed please let me know.
The text was updated successfully, but these errors were encountered:
it's not important, they don't use labels in train mode and val mode.
Sorry, something went wrong.
No branches or pull requests
I really appreciate your work and thanks a lot for sharing this work with us!
I am now following your code step by step to understand the model and apply it to my data.
And while I'm doing this, I got a question.
In the line 187-192 of data_loader.py
The code is written like this.
187 def getitem(self, index):
***
190 return np.float32(self.train[index:index + self.win_size]), np.float32(self.test_labels[0:self.win_size])
192 return np.float32(self.val[index:index + self.win_size]), np.float32(self.test_labels[0:self.win_size])
Shouldn't the '0' be changed to 'index'?
self.test_labels[0:self.win_size] -> self.test_labels[index:index + self.win_size]
If there's anything that I missed please let me know.
The text was updated successfully, but these errors were encountered: