aimhubio / aim

Aim 💫 — An easy-to-use & supercharged open-source experiment tracker.
https://aimstack.io
Apache License 2.0
4.94k stars 299 forks source link

pin SQLAlchemy to <2.0 #3039

Closed xquyvu closed 6 months ago

xquyvu commented 9 months ago

I'm getting this warning when having aim installed.

E sqlalchemy.exc.MovedIn20Warning: Deprecated API features detected! These feature(s) are not compatible with SQLAlchemy 2.0. To prevent incompatible upgrades prior to updating applications, ensure requirements files are pinned to "sqlalchemy<2.0". Set environment variable SQLALCHEMY_WARN_20=1 to show all deprecation warnings. Set environment variable SQLALCHEMY_SILENCE_UBER_WARNING=1 to silence this message. (Background on SQLAlchemy 2.0 at: https://sqlalche.me/e/b8d9)

Although there's an open issue for SQLAlchemy 2.0 support: #2514, it doesn't look like it's being prioritised. In the mean time, let's pin the dependency to silence this warning.

In the short term, my team's configuration treats DeprecationWarnings as Error, which means I cannot run any tests with VSCode testing when having aim installed.

Thanks!

CLAassistant commented 9 months ago

CLA assistant check
All committers have signed the CLA.

mihran113 commented 6 months ago

Hey @xquyvu! Thanks a lot for the PR and sorry for late response. As we've already addressed the #2514 issue, I'm going to close this PR.