Skip to content

assignment_10 #3

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

assignment_10 #3

wants to merge 5 commits into from

Conversation

thervig
Copy link

@thervig thervig commented Apr 20, 2016

For this assignment I chose to create a new Tweet Class. I wasn't comfortable using inheritance from the Point Class for the twitter data since I believe a tweet has more attributes than a simple point. Maybe after growing more comfortable with Python I can learn to combine the tweet and point classes to make simpler code.

@thervig
Copy link
Author

thervig commented Apr 20, 2016

I had some trouble with how to open and display the tweets in my script..I think I can figure it out I just ran out of time tonight and will come back to it later.

self.setGeometry(300, 300, 700, 700)
self.setWindowTitle('Title Window')
self.show()

Copy link
Contributor

Choose a reason for hiding this comment

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

Here is where you would add an open method (or wire line 34 to the open_tweets function on line 50.) After you open the tweets, you could all the show_tweets methods. The pieces are all here (show_tweets would need added logic for lat/lon) just a connecting issue.

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