You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hi, I noticed some missing arguments in EnvArgs for instantiating BrowserGym with full configurations.
For example, timeout is not configurable in EnvArgs, and the timeout for context or page configuration can only be set by the attribute in the Task class this way.
As well as pw_chromium_kwargs, which is not that important but still incomplete for designing EnvArgs.
Hopefully, this is useful for your improvement! I've been deeply reading and reconstructing your codebase to do my experiments; I love your work, and thank you for continued updating:)
The text was updated successfully, but these errors were encountered:
I'm not sure why we didn't include timeout in EnvArgs, it should be pretty easy to add. @recursix was there a reason why we didn't have it? Should we include all BrowserEnv parameters in EnvArgs ?
Hi, I noticed some missing arguments in EnvArgs for instantiating BrowserGym with full configurations.
For example,
timeout
is not configurable in EnvArgs, and the timeout for context or page configuration can only be set by the attribute in the Task class this way.As well as
pw_chromium_kwargs
, which is not that important but still incomplete for designing EnvArgs.Hopefully, this is useful for your improvement! I've been deeply reading and reconstructing your codebase to do my experiments; I love your work, and thank you for continued updating:)
The text was updated successfully, but these errors were encountered: