-
-
Notifications
You must be signed in to change notification settings - Fork 80
Fix [FutureWarning: The frame.append method is deprecated and will be removed from pandas in a future version. Use pandas.concat instead.] #247
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
Conversation
… removed from pandas in a future version. Use pandas.concat instead.]
@psujit775 could you please check why the tests are failing? Maybe the pandas version needs to be updated |
@4n4nd updated Pipfile and requirements.txt to pandas min version >=1.4.0. Deprecations reference: https://pandas.pydata.org/docs/whatsnew/v1.4.0.html#whatsnew-140-deprecations-frame-series-append |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Once all the changes are done, would you mind squashing the commits? Otherwise it looks good to me!
/lgtm 🚀
@psujit775 could you also update the piplock? |
@4n4nd updated the Pipfile.lock file. |
Thanks! |
Thank you Anand for merging it.
I know it's just a little change but I'm so happy.
Thanks again,
…On Thu, Oct 20, 2022 at 10:24 PM Anand Sanmukhani ***@***.***> wrote:
Thanks!
—
Reply to this email directly, view it on GitHub
<#247 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AMC2IGT7WPM7NUBPQUCGG6DWEF2LDANCNFSM6AAAAAAQBNTRAQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Thank you for working on it! |
Fix [FutureWarning: The frame.append method is deprecated and will be removed from pandas in a future version. Use pandas.concat instead.]
Upstream issue: pandas-dev/pandas#35407