ray-project / xgboost_ray

Distributed XGBoost on Ray
Apache License 2.0
143 stars 34 forks source link

[Private APIs] Switch to `ray.worker.get_resource_ids` #89

Closed ijrsvt closed 3 years ago

ijrsvt commented 3 years ago
krfricke commented 3 years ago

Thanks @ijrsvt. We'll need to keep this backwards compatible with the latest Ray release, so it might be best to move the get_resource_ids to a separate function until the change is reflected in the next release. I'm happy to update your PR with that.

ijrsvt commented 3 years ago

@krfricke using the private API works on all versions of Ray. This is where the function is actually defined :)

It was just imported to the top level ray.get_resource_ids

krfricke commented 3 years ago

Ah sorry, I mixed that up - thanks!