Get mask in compute_histogram with call that also tries join_on_key

[jfoster17]

Pull Request Template

Description

This is a single-line change to use a method that also tries join_on_key inside compute_histogram. Without this change, 2D scatter plots that are in density_map mode will not show selections that should propagate through join_on_key type connections. compute_statistic in data.py might also not working correctly with join_on_key because it calls subset_state.to_mask(self, view)-- but it also calls subset_state.to_array(self, cid) and so this would be a larger fix (if it even is a problem).

[codecov]

Codecov Report

Merging #2242 (d0a1f20) into main (112809a) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

❗ Current head d0a1f20 differs from pull request most recent head a1ee97b. Consider uploading reports for the commit a1ee97b to get more accurate results

@@            Coverage Diff             @@
##             main    #2242      +/-   ##
==========================================
- Coverage   88.04%   88.01%   -0.04%     
==========================================
  Files         247      247              
  Lines       23172    23172              
==========================================
- Hits        20402    20394       -8     
- Misses       2770     2778       +8     

Impacted Files	Coverage Δ
glue/core/data.py	91.37% <100.00%> (ø)
glue/utils/qt/app.py	90.00% <0.00%> (-6.00%)	⬇️
glue/app/qt/preferences.py	88.61% <0.00%> (-2.44%)	⬇️
glue/plugins/dendro_viewer/data_factory.py	97.72% <0.00%> (-2.28%)	⬇️
glue/utils/qt/helpers.py	53.93% <0.00%> (-1.13%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 112809a...a1ee97b. Read the comment docs.

[astrofrog]

Thanks, this looks good!