Skip to content

Refactoring: DataFrameWidget(QWidget) -> DataFrameWidget(QTableView) #7021

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 4 commits into from
Closed

Refactoring: DataFrameWidget(QWidget) -> DataFrameWidget(QTableView) #7021

wants to merge 4 commits into from

Conversation

rominf
Copy link

@rominf rominf commented May 1, 2014

I think that deriving DataFrameWidget from the QTableView is more logical than from QWidget

@jreback
Copy link
Contributor

jreback commented May 1, 2014

did you open 2 PR's fro this? #7021, #7020 pls close one of them

here are some git tips: https://github.com/pydata/pandas/wiki/Using-Git

@rominf rominf changed the title PyQt 5 support added Refactoring: DataFrameWidget(QWidget) -> DataFrameWidget(QTableView) May 1, 2014
@rominf
Copy link
Author

rominf commented May 1, 2014

Nope. I don't know why but when I did git pull-request via GitHub hub tool it copied the old message. In this pull request I refactoried some code.

Or I should wait until my first PR will be accepted? I'm not an expert in git yet...

@jreback
Copy link
Contributor

jreback commented May 1, 2014

ok, before you go further we need to create a test structure. That is important to retain backward compatibility.

@rominf
Copy link
Author

rominf commented May 1, 2014

I agree with you but I need to submit labs in mathematical modeling 😃 So I'll write my code. When someone would get ideas how to unit test PyQt application I'll add tests.

@jreback
Copy link
Contributor

jreback commented May 1, 2014

@jreback jreback added QT labels May 1, 2014
@jreback jreback added this to the 0.15.0 milestone May 1, 2014
@rominf
Copy link
Author

rominf commented May 1, 2014

OK, I'll try it later. I don't know how to deal with constantly changing version of PyQt. May be we should just install PyQt from repositories? I don't know how does Travis work but I think it should be possible. Other problem is: PySide, PyQt4 and PyQt5 should not be mixed. I don't know how to make several build environments.

@jreback
Copy link
Contributor

jreback commented May 1, 2014

they can be done on different builds. All I need are instruction to install from repositories and/or pip.

@jreback
Copy link
Contributor

jreback commented Jan 25, 2015

need tests for these, maybe should be taken out of pandas

@jreback jreback closed this Jan 25, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Output-Formatting __repr__ of pandas objects, to_string Visualization plotting
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants