NREL / flasc

A rich floris-driven suite for SCADA analysis
https://nrel.github.io/flasc/
BSD 3-Clause "New" or "Revised" License
30 stars 18 forks source link

Upgrade sqlalchemy requirement to v2 #160

Closed paulf81 closed 7 months ago

paulf81 commented 7 months ago

Upgrade sqlalchemy requirement to v2

This pull request addresses Issue #159. Previously there had been some errors when using sqlalchemy v2, but raising the requirement and testing the code for both reading and writing to postgres shows no issue so implementing this pull request to raise the requirement to ~=2.0. The pull request also fixes some small issues in sqldatabase_management.py identified in testing. Since this is a very small change, directly merging (hope that is ok @misi9170 !)