-
Notifications
You must be signed in to change notification settings - Fork 1
Open
Description
Unfortunately for external usability, this repo relies a lot on relative pathing. If I just want to use the benchmark as a submodule for a different project I first need to manually add the directory to my path since it's not an installable package. Then I need to switch the data path since that is hardcoded to be relative for some reason.
Suggested improvements:
- Make the repo an installable package to make it importable
- Instead of reading from a relative path, hardcode the default to be absolute (can be done e.g. by using Path(file).resolve().parent as a base path). Then the directory from the repo will always be the default independent of the execution location
Metadata
Metadata
Assignees
Labels
No labels