jay
jay
> @Jay-ju my main concern is that when looking at the list of available functions, via IDE tools, properties like the width, height, etc won't show up because they are...
@universalmind303 Can this be merged?？
> @Jay-ju can you fix the style issue on CI? @srilman fixed it
@richardliaw Help me review this PR when it's convenient？
@colin-ho Do you have time to help check the function of this ut? It seems unnecessary.
Error messge for gpu env ``` return await self.task E ^^^^^^^^^^^^^^^ E File "/data00/code/Daft/daft/runners/flotilla.py", line 140, in _get_result E raise e E File "/data00/code/Daft/daft/runners/flotilla.py", line 124, in _get_result E metadatas...
> Hi @Jay-ju, I'm not sure what the intention behind this PR is. Calling `.override_options(num_gpus=None)` should reset the GPU resources to the default of zero, which is what that parameter...
> > @kevinzwang OK. I'm quite curious why we need to reset the number of detectable GPUs here? It seems that if this is the expectation, the current implementation can't...
> > If it is intended that when it equals None, it should be 0, then the environment variable CUDA_VISIBLE_DEVICES would be modified, but this is not currently observed. >...
@colin-ho Thank you very much. I later looked at the code and saw this too. I feel that if we have to handle it according to 1 here, we will...