marius-team / marius

Large scale graph learning on a single machine.
https://marius-project.org
Apache License 2.0
160 stars 45 forks source link

Remove output_directory as a required argument #78

Closed AnzeXie closed 2 years ago

AnzeXie commented 3 years ago

Is your feature request related to a problem? Please describe. Currently, we require the user to specify the output_directory.

We want to remove this requirement and set the output directory to match the name of the dataset by default. For custom datasets, we should choose a reasonable default name such as “custom_dataset/”

Users should still be able to specify the output directory if they wish.

Describe the solution you'd like We remove the output_directory as a required argument. For built-in dataset, we set the default base directory name "_dataset", for custom datasets, we set the default base directory name "custom_dataset".

Users are given the option to specify the output directory name if they want.

Additional context This issue corresponds to MAR-51 (https://marius-project.atlassian.net/browse/MAR-51).