Fix processing dir default - Githubissues

VisionSystemsInc / terra

Terra - Run your algorithm anywhere on earth

MIT License

0 stars 3 forks source link

Fix processing dir default #52

Closed andyneff closed 4 years ago

andyneff commented 4 years ago

The code for using settings.terra.config_file to determine settings.processing_dir was not working due to a chicken-egg issue.

Call _setup

Calls configure
Send signal
- Logger receives signal, and start setting up logger
- Needs to know where to put log files, so check settings.processing_dir
- settings.processing_dir looks for setttings.terra.config_file, doesn't see it yet
Returns to _setup
Then set settings.terra.config_file, but it's too late

The solution was essentially to use a global/singleton variable 😞 to get around this issue.

andyneff commented 4 years ago

Merge after #51

Fixes https://bitbucket.org/visionsystemsinc/terra_dsm/pull-requests/203