explicit-comms: don't mix - and _ in config

[madsbk]

Using dask.config.get("explicit_comms-batchsize", 1) doesn't read DASK_EXPLICIT_COMMS_BATCHSIZE correctly.

[codecov-commenter]

Codecov Report

Base: 87.90% // Head: 87.91% // Increases project coverage by +0.01% 🎉

Coverage data is based on head (39d88f5) compared to base (66a6a46).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files

@@               Coverage Diff                @@
##           branch-23.02    #1096      +/-   ##
================================================
+ Coverage         87.90%   87.91%   +0.01%     
================================================
  Files                17       17              
  Lines              2298     2300       +2     
================================================
+ Hits               2020     2022       +2     
  Misses              278      278              

Impacted Files	Coverage Δ
dask_cuda/cli.py	86.40% <ø> (ø)
dask_cuda/disk_io.py	56.19% <ø> (ø)
dask_cuda/initialize.py	87.23% <ø> (ø)
dask_cuda/is_spillable_object.py	84.37% <ø> (ø)
dask_cuda/proxy_object.py	90.52% <ø> (ø)
dask_cuda/utils.py	83.47% <ø> (ø)
dask_cuda/explicit_comms/dataframe/shuffle.py	98.69% <100.00%> (+0.01%)	⬆️
dask_cuda/proxify_device_objects.py	97.32% <100.00%> (ø)
dask_cuda/proxify_host_file.py	93.71% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[pentschev]

LGTM @madsbk , but I think there's room to improve how explicit comms are organized. WDYT?

[pentschev]

Given there are multiple properties for explicit-comms (i.e., explicit-comms and explicit-comms-batchsize), I think it would make sense to make that a collection, e.g., explicit-comms.enable and explicit-comms.batchsize, or something analogous.

[wence-]

I think I advocated for this in a previous PR, so I am in favour of this idea.

[madsbk]

Heh, yes @wence- had the exact same suggestion: #1068 (comment) :)

I will change to explicit-comms.enable in a follow up PR

[madsbk]

/merge