minorleaguegrayonassociates / GrayonSlam

3 stars 1 forks source link

Add distances to database #48

Closed Roger-Chavez closed 5 years ago

Roger-Chavez commented 5 years ago

Closes #(issue)

Key features

Notes

Roger-Chavez commented 5 years ago

Where is the graph data-structure? Why so many CSV file changes? There are more changes to them than necessary, and you didn't provide a description in the PR description about why you made them.

I thought we agreed I was adding the distance data to the database, mainly what I wanted to do was just add the coordinates, but figured I'd add the distance data while I was at it. Need the coordinates for the main pr I'm trying to land by Monday. We never agreed on adding a graph, all I was adding was the distances so that when the graph is added it can just grab the distances from the database. The only things that changed form the CSV were the coordinates, updating the stadium ids to increment with the team and we also agreed to treat the edges as undirected so removed duplicate edges. I originally put the ids on distances and the id's for the MLBInfo were out of order, I went back and redid it. None of the data that Lebo provided was altered.