-
Notifications
You must be signed in to change notification settings - Fork 49
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support for sqlalchemy-mate 2.0.0.1 #79
Comments
As a workaround until uszipcode is updated, you can pin |
i am getting following error
|
Heh sorry, did that comment a bit too quickly. I had a typo. It has been edited. |
Hi folks and @bdipietro since sqlalchemy 2.X has breaking changes, the sqlalchemy_mate 2.X won't be able to compatible with this library. You can add I am working on this to fully adapt sqlalchemy 2.X API, however, it may introduce breaking change in thank you for your help answering question @mdelcambre |
Hi, I reverted sqlalchemy to 2.0.0.0 and it works.
Thanks!
Joy
…On Wed, May 22, 2024 at 11:16 PM Sanhe Hu ***@***.***> wrote:
Hi folks and @bdipietro <https://github.com/bdipietro> since sqlalchemy
2.X has breaking changes, the sqlalchemy_mate 2.X won't be able to
compatible with this library.
You can add sqlalchemy_mate>=1.4.28.3,<2.0.0.0 in your requirements file
to fix this issue.
I am working on this to fully adapt sqlalchemy 2.X API, however, it may
introduce breaking change in uszipcode as well, let me figure out a more
reasonable way to minimize the impact of the compatibility.
thank you for your help answering question @mdelcambre
<https://github.com/mdelcambre>
—
Reply to this email directly, view it on GitHub
<#79 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ARGEMR3FI6FC7HJLUE7JX2LZDVUSZAVCNFSM6AAAAABH4HI7R6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRWGE4TQNRTGU>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
@joyjitroy also, the data is still based on census 2010. I found a good resources to crawl 2020 data. I need some time to upgrade this library using the latest data. |
Great sir! Please keep me posted. I’m my code I found 36 zip code without
any county code. I marked them unknown,
…On Thu, May 23, 2024 at 10:02 AM Sanhe Hu ***@***.***> wrote:
@joyjitroy <https://github.com/joyjitroy> also, the data is still based
on census 2010. I found a good resources to crawl 2020 data. I need some
time to upgrade this library using the latest data.
—
Reply to this email directly, view it on GitHub
<#79 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ARGEMR4NFS7FUQTQPU4T2HLZDYAJZAVCNFSM6AAAAABH4HI7R6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRXGM3DQNZTGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
This is currently broken out of the box with a pip install. Sure, this can be worked around by also somehow magically knowing to install Then the real fix with potential major changes can happen at a later time. |
* Update TOS * Update privacy policy, use absolute url to resolve to the remove data page. * Add in missing load of absolute tag. * Pin back MacHu-GWU/uszipcode-project#79
Describe the problem
During import the uszipcode package fails with the following error.
module 'sqlalchemy_mate' has no attribute 'ExtendedBase'
A dependent package called
sqlalchemy-mate
updated to version 2.0.0.1 last night and theExtendedBase
attribute is now undersqlalchemy_mate.api
Sample code to reproduce the problem
When executing:
import uszipcode
The following command within
model.py
errors:class AbstractSimpleZipcode(Base, sam.ExtendedBase):
Potential Solution
Update
uszipcode
to leverage theExtendedBase
attribute fromsqlalchemy_mate.api
in version 2.0.0.1.Maybe you can try talk to the author lively here https://gitter.im/MacHu-GWU-Python-Library-Technical-Support/community
The text was updated successfully, but these errors were encountered: