Skip to content

implemented description property #574

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

Closed
wants to merge 1 commit into from

Conversation

joonro
Copy link

@joonro joonro commented Jan 4, 2012

Hi,

This is a basic implementation of description and unit properties in DataFrame.
https://github.com/wesm/pandas/issues/39

I used dict to store both description and unit. I thought about using tuples but dict would be better if a user wants to modify individual descripition/unit.

One can directly access description/unit, and also DataFrame.info() will print out those if they exist.

I haven't touched at the storing part yet so it is very rudimentary now (no way to save the descriptions), but I wanted to know what you think. Please let me know.

And also, is this (topic branch) right way to submit a pull request? I'm still learning about git. :)

Happy new year!
-Joon

@wesm
Copy link
Member

wesm commented Jan 4, 2012

Yes this is correct git usage :)

The problem with this is that as soon as you do anything with the DataFrame you've created the column metadata is going to be lost. Probably the right way to do it would be to store the information inside the column index itself. So it's a bit of an involved undertaking. I will take a closer look at your patch and think about how much work it would be to do

@joonro
Copy link
Author

joonro commented Jan 4, 2012

That makes sense. Let me know!

@wesm
Copy link
Member

wesm commented Sep 9, 2012

Hey sorry this PR has been open for so long. I'm going to close it until more work is put into a deeper solution for tracking metadata. It's a pretty big project I think

@wesm wesm closed this Sep 9, 2012
dan-nadler pushed a commit to dan-nadler/pandas that referenced this pull request Sep 23, 2019
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