Skip to content
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

Create empty dataframe from Pandas DataFrame Model #1880

Merged
merged 5 commits into from
Dec 31, 2024

Conversation

mamo3gr
Copy link
Contributor

@mamo3gr mamo3gr commented Dec 20, 2024

I'm interested in #1039 however it has not been merged. I would like to salvage the PR partially (temporarily), that is, only for DataFrame Model.

@mamo3gr mamo3gr force-pushed the feature/create-empty-dataframe branch 2 times, most recently from 8fd96a3 to c54b556 Compare December 20, 2024 07:09
@mamo3gr mamo3gr marked this pull request as draft December 27, 2024 13:06
Signed-off-by: Mamoru Miura <mamo3gr@gmail.com>
@mamo3gr mamo3gr force-pushed the feature/create-empty-dataframe branch from c54b556 to 918abf7 Compare December 27, 2024 15:40
Copy link

codecov bot commented Dec 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.26%. Comparing base (812b2a8) to head (4ef1bc7).
Report is 185 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1880      +/-   ##
==========================================
- Coverage   94.28%   93.26%   -1.02%     
==========================================
  Files          91      121      +30     
  Lines        7013     9299    +2286     
==========================================
+ Hits         6612     8673    +2061     
- Misses        401      626     +225     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@@ -20,6 +20,8 @@
cast,
)

import pandas as pd
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the generic dataframe.model module should not depend on any dataframe library, can you move this to https://github.com/unionai-oss/pandera/blob/main/pandera/api/pandas/model.py?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure. I have done it with 38a5a95.

Signed-off-by: Mamoru Miura <mamo3gr@gmail.com>
Signed-off-by: Mamoru Miura <mamo3gr@gmail.com>
@mamo3gr mamo3gr force-pushed the feature/create-empty-dataframe branch from 11aef4d to fb1ac6a Compare December 27, 2024 22:05
@mamo3gr mamo3gr changed the title Create empty dataframe from DataFrame Model Create empty dataframe from Pandas DataFrame Model Dec 27, 2024
@mamo3gr mamo3gr marked this pull request as ready for review December 28, 2024 13:10
@mamo3gr mamo3gr requested a review from cosmicBboy December 28, 2024 13:10
@mamo3gr mamo3gr force-pushed the feature/create-empty-dataframe branch from cfef591 to d4987ac Compare December 29, 2024 00:34
Signed-off-by: Mamoru Miura <mamo3gr@gmail.com>
@mamo3gr mamo3gr force-pushed the feature/create-empty-dataframe branch from d4987ac to bd42149 Compare December 29, 2024 00:34
@cosmicBboy
Copy link
Collaborator

thanks @mamo3gr ! 🚀

@cosmicBboy cosmicBboy merged commit 1b8e925 into unionai-oss:main Dec 31, 2024
145 of 146 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants