vis cleanup

[gipert]

• type hints and docstring updates
• basic WaveformBrowser test

[codecov-commenter]

Codecov Report

Merging #312 (7f2c80d) into main (16a3246) will increase coverage by 12.22%.
The diff coverage is 77.27%.

@@             Coverage Diff             @@
##             main     #312       +/-   ##
===========================================
+ Coverage   33.10%   45.32%   +12.22%     
===========================================
  Files          73       73               
  Lines        6187     6186        -1     
===========================================
+ Hits         2048     2804      +756     
+ Misses       4139     3382      -757     

Impacted Files	Coverage Δ
src/pygama/vis/waveform_browser.py	67.39% <76.19%> (+67.39%)	⬆️
src/pygama/vis/__init__.py	100.00% <100.00%> (+100.00%)	⬆️
src/pygama/lgdo/lh5_store.py	76.92% <0.00%> (+1.13%)	⬆️
src/pygama/lgdo/table.py	95.31% <0.00%> (+1.56%)	⬆️
src/pygama/dsp/_processors/convolutions.py	6.97% <0.00%> (+6.97%)	⬆️
src/pygama/lgdo/waveform_table.py	88.00% <0.00%> (+7.00%)	⬆️
src/pygama/dsp/_processors/gaussian_filter1d.py	10.34% <0.00%> (+10.34%)	⬆️
src/pygama/dsp/_processors/wiener_filter.py	11.11% <0.00%> (+11.11%)	⬆️
src/pygama/dsp/_processors/trap_filters.py	12.50% <0.00%> (+12.50%)	⬆️
.../pygama/dsp/_processors/get_multi_local_extrema.py	12.82% <0.00%> (+12.82%)	⬆️
... and 22 more

---

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 16a3246...7f2c80d. Read the comment docs.

[jasondet]

Does this overlap with PR 310?

Also, should be on main not refactor.

[jasondet]

@iguinn also please review this one.

[iguinn]

I reviewed the annotations and comment changes to waveform_browser, and aside from the comment it looks good to me.

[jasondet]

@gipert can you check why Build documentation is failing? I couldn't figure it out from the Details.

[jasondet]

The build_documentation failure is a sphinx-doc/sphinx#10701. I have turned off this requirement for merging and will merge.