search

microsoft / tf-gnn-samples

TensorFlow implementations of Graph Neural Networks
MIT License
914 stars 229 forks source link

exception in varmisuse task ,when use no_parallel:bool=True in _load_data() method #3

Closed SongXueZhi closed 4 years ago

SongXueZhi commented 4 years ago

when use params "no_parallel=True" in _load_data() ,We can get the data successfully from load_single_sample method , and reurn result list[GraphSample].but __load_data() can't get the result from “return _load_data()” ,or the type of the result that should be a list is a generator. the code as follow: image the exception: image the debug info in vscode: image image In order to facilitate debugging, we will assign the result to the temporary variable 'result' first.It is easy to find that there is correct data in ‘result’, but there is no data in self.loaded_data

mmjb commented 4 years ago

Sorry for the late reply. This is now fixed in 8732bf2, which makes the parallel and non-parallel versions both use a generator - I haven't really figured out why the existing code didn't work, but I've verified that this new version works fine (for me).