SQLite Fail on threading about thermo

[cloasdata]

Meanwhile I created a pull request. #141

[CalebBell]

Hi @cloasdata,
Interesting find! I hadn't encountered this issue before. All use of sqlite in thermo is currently read-only so the check_same_thread=False option is safe globally.

I appreciate your willingness to write a PR. In the spirit of keeping things simple, can you please reduce its scope to only that change without allowing arbitrary user options and having a test? I think this is best solved with the simple boolean change.

Sincerely,
Caleb

[cloasdata]

Hello Caleb,

Understood.
I have updated the PR as requested. The test now uses threading explicit to see if an exception was thrown in the child thread.
I found this error while implementing dearpygui, a gui framework. DPG uses separate threads for state and callback management. And there this problems occurs. Sometimes you have luck and the callback is always done in the same thread but sometimes not and you get this weird error.
cheers Simon

[CalebBell]

Hello Simon,
That's cool to hear. Thanks for this fix! It'll be in the next release.
Sincerely,
Caleb