-
Notifications
You must be signed in to change notification settings - Fork 53
[GNNFlux] Translate Traffic prediction
Pluto notebook to Literate
#572
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
Conversation
|
||
# ## Dataset: METR-LA | ||
|
||
# We use the `METR-LA` dataset from the paper [Diffusion Convolutional Recurrent Neural Network: Data-driven Traffic Forecasting](https://arxiv.org/pdf/1707.01926.pdf), which contains traffic data from loop detectors in the highway of Los Angeles County. The dataset contains traffic speed data from March 1, 2012 to June 30, 2012. The data is collected every 5 minutes, resulting in 12 observations per hour, from 207 sensors. Each sensor is a node in the graph, and the edges represent the distances between the sensors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does it mean the edges represent the distances between the sensors
? should be clarified
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
better something like: the edge weights are the distances between the sensor locations?
train_loader = zip(features[1:200], targets[1:200]); | ||
test_loader = zip(features[2001:2288], targets[2001:2288]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
motivate this choice of ranges
GraphNeuralNetworks/docs/src_tutorials/introductory_tutorials/traffic_prediction.jl
Show resolved
Hide resolved
model = GNNChain(TGCN(2 => 100; add_self_loops = false), Dense(100, 1)) | ||
|
||
#  | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
here it would be useful to show the output of the model and how it is interpreted as a prediction
* Fix temporal graph classification literate * [GNNFlux] Translate `Traffic prediction` Pluto notebook to Literate (#572) * Add traffic prediction * Fixes * Fix temporal graph classification literate * Back to Vector * Fixes * Add info about 250 graphs
#545