-
Notifications
You must be signed in to change notification settings - Fork 230
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
Added TAG and SCREENSHOT options #137
base: master
Are you sure you want to change the base?
Conversation
currenttime = str(int(time.time())) | ||
|
||
self.app.primaryScreen().grabWindow(self.app.desktop().winId()).save(os.path.join(self.directory,currenttime+'.png'), 'png') | ||
print "Screenshot saved to "+ os.path.join(self.directory,currenttime+".png") |
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.
Should use logging
and print
as a function (for Python 3).
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.
Actually the project only compiles with python2
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.
Not really an argument, is it? :)
(it will only make it more difficult to port in the future)
Apart from that: what about using logging instead?
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.
There's no actual log file on selfspy, so I don't see how this would be useful. Anyways I'm willing to reconsider
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.
It would be logged to the default output handler (console).
FWIW, I found this useful when looking at automatically taking screenshots: http://www.autopy.org/documentation/api-reference/bitmap.html#capture_screen |
@blueyed why should I use that instead of PyQt? |
I just wanted to leave this as some suggestion - it might be more lightweight than PyQt. |
@Hacktivista I'm back doing some cleaning. I think that the PyQT-dependency should be optional (especially since I'm guessing that this only works on Linux?), if people want the screenshot functionality. I don't know what the normal method is for letting someone take over a project. Tell me if you know, otherwise I will have to look it up. |
Should work on every OS. Since it's QT does. Although I proved on Linux only. Here into to transfer repo: https://help.github.com/articles/transferring-a-repository/ |
@Hacktivista I tried the transfer, but since you already have a fork named "selfspy", it won't go through. You'll have to rename or remove it first! |
Now you can set a TAG and capture SCREENSHOTS while using selfspy.
PS: Gurgeh, I'm the guy which contacted you about the project through Twitter, how can we manage the retake? Greetings!