TensorVision / TensorVision

A simple toolkit to easily apply image classification networks to your data.
MIT License
119 stars 45 forks source link

Relative paths in hypes #53

Open MartinThoma opened 8 years ago

MartinThoma commented 8 years ago

Currently, relative paths in hypes have to get handled by every single script. This is an issue, because we (or the people using the loaded hypes dict) could simply forget that the relative paths are not relative to the hypes.json file, but relative to the calling Python script.

We handled those here:

I suggest to automatically make paths absolute in the hypes loading function (see #52). For example, we could say that each key which ends with _path will be made absolute if it is relative.

MarvinTeichmann commented 8 years ago

Sounds like a good idea!

MarvinTeichmann commented 8 years ago

Just saw an issue with this approach: What kind of data should be loaded depends on the content. I.e. data is assumed to be relative to TV_PATH_DATA, while models should be defined relative to hypes file.