openai / gym-http-api

API to access OpenAI Gym from other languages via HTTP
MIT License
293 stars 142 forks source link

Update Rust Client #56

Open NivenT opened 7 years ago

NivenT commented 7 years ago

I updated the rust client to use reqwest instead of directly using hyper, fixed a bug in the way box spaces were being sample, and just all around made the code a little nicer. I believe this should resolve #55

Also, I went ahead and fixed the contributor's page issue in the readme.

unixpickle commented 7 years ago

Is this a breaking change? I.e., will existing code that uses the old API break with this change?

NivenT commented 7 years ago

Yes it would, but minimally. GymClient::new now returns a Result, so users would not need to handle the case that an error occurs.