Add a modified version of the auto MPG dataset #1190
Conversation
bloebp
force-pushed
the
add_auto_mpg_dataset
branch
from
85d98a3
to
e9d6d3d
Compare
There was a problem hiding this comment.
thanks for adding this, @bloebp. Right now, the license file is in the root directory with many other example notebooks. Would it be cleaner to create a new directory called datasets/auto-mpg or and then add both the csv and license file in that sub-folder? The only code change will be to specify the correct path in the notebook.
Before, we were loading it directly from the UCI dataset repository. However, this cause connection issues and build fails. The dataset is now added directly with its respective license. Further, removed the output of the causal discovery example notebook to ensure that it gets compiled while building. Signed-off-by: Patrick Bloebaum <bloebp@amazon.com>
bloebp
force-pushed
the
add_auto_mpg_dataset
branch
from
e9d6d3d
to
00ff0a3
Compare
Makes sense! We probably should even move all datasets into that folder. But that should be a follow-up PR. |
Before, we were loading it directly from the UCI dataset repository. However, this cause connection issues and build fails. The dataset is now added directly with its respective license.
Further, removed the output of the causal discovery example notebook to ensure that it gets compiled while building.