-
Notifications
You must be signed in to change notification settings - Fork 41
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
Add other browsers #175
Comments
@marco-c i had a couple of doubts regarding this. Is this the same as adding a webdriver for a particular browser and then updating collect.py so that this driver is also run alongside chrome and firefox ? Apart from that , after we do add a browser won't we have to update data_inconsitencies.py and some functions in utils.py to accommodate these new set of images? |
Yes, we will have to add a new webdriver and use it in collect.py and update all the places where we were just expecting Firefox and Chrome. |
So should we add a task list in this Issue ? |
@marco-c having taken a look into it, I've found that safari and edge both are dependent on their respective os and cannot be run cross-platform. |
Opera uses Chromium, so there will be no difference between Opera and Chrome. |
safari is available for windows. So windows supports edge and safari both officially. We can also use wine if we want to use it in linux imo. |
Is it still available? I thought they stopped supporting Windows. |
I've tried to run Safari through selenium on Windows , they use an extension which is only compatible with Safari installed on a MacOs. |
Although selenium supports safaridriver in windows, Apple has dropped support :( so It might not work properly. Sorry for the confusion. I think checking the os and choosing edge or safari based on that should be the way to go. |
Probably not so valuable to add Edge after all: https://blog.mozilla.org/blog/2018/12/06/goodbye-edge/ 😢 But maybe we can use it anyway so we have more diverse screenshots and we're more likely to have incompatibilities in our training set. |
The more browsers we have, the more screenshots we can collect
The text was updated successfully, but these errors were encountered: