-
Notifications
You must be signed in to change notification settings - Fork 16
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
Decouple curve creation from driver viewport logic #10
Comments
I assume by human curve logic, you meant Also, only WebCursor takes in the driver argument. SystemCursor does not. But however, i'm open to any pull requests that can add value to the project. Keep in mind that this is very unreadable, poorly structured code and not actively maintained anymore. Good luck and thank you! |
I could be mistaken, but it looks like Can you explain what's the intent behind restricting the points when generating for the web? |
The tweaking and modifications (including this restricting you mentioned) for the WebCursor, are just my adaptions to make it work, because the ActionChains module of Selenium that I use for actually moving the cursor isn't customizable at all. I forgot the actual reason, but I couldn't control the duration/speed of the movement at all on the web, and the only solution I found was that : the less points there are => the faster the movement will be, the more points there are => the slower the movement will be. The correct path was to use a lower-level function that I could build on myself, rather than using ActionChains, but I was a novice back then so... As for using a generalized version of |
I want to use the curve points generated, for controlling mouse movements while scraping with scrapy-playwright. |
If you can do this: |
I would like to use this project for it's human curve logic only, and plumb that into my own use case, however the current API is structured to always take in a Driver. I could submit a PR if you're open to this.
The text was updated successfully, but these errors were encountered: