Improve wsinterp docstrings and test function

[bobleesj]

No description provided.

[github-actions]

Warning! No news item is found for this PR. If this is a user-facing change/feature/fix,
please add a news item by copying the format from news/TEMPLATE.rst.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.69%. Comparing base (5f67f6c) to head (f737a13).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #221   +/-   ##
=======================================
  Coverage   99.69%   99.69%           
=======================================
  Files           8        8           
  Lines         325      325           
=======================================
  Hits          324      324           
  Misses          1        1           

Files with missing lines	Coverage Δ
tests/test_resample.py	94.73% <100.00%> (ø)

[bobleesj]

Ready for review - pls see my in-line comments.

[bobleesj]

@sbillinge ready for review! - added DOI, didn't add any additional empty liens

[bobleesj]

@sbillinge ready for review! - added DOI, didn't add any additional empty liens

@sbillinge it passes - just codecov..

[sbillinge]

This looks great, o can merge when it passes.

Nor I am greedy though..... Would it be nice if it could take, optionally, a DiffractionObject and return a distraction object? Or maybe it is better to make a method on the DO itself?

[bobleesj]

@sbillinge v5 has been released. Shall we migrate from our current v4 to v5 for Codecov actions?

https://github.com/codecov/codecov-action

[bobleesj]

Nor I am greedy though..... Would it be nice if it could take, optionally, a DiffractionObject and return a distraction object? Or maybe it is better to make a method on the DO itself?

having wsinterp as a modular and standalone function, and then have DiffractionObject integrate wsinterp seems like a scalable approach to me imo?

[sbillinge]

Nor I am greedy though..... Would it be nice if it could take, optionally, a DiffractionObject and return a distraction object? Or maybe it is better to make a method on the DO itself?

having wsinterp as a modular and standalone function, and then have DiffractionObject integrate wsinterp seems like a scalable approach to me imo?

Yah my thinking too. Actually it is for pdfs, not for diffraction, but for sure there is a PDFObject in our future.

[bobleesj]

Yah my thinking too. Actually it is for pdfs, not for diffraction, but for sure there is a PDFObject in our future.

I left a comment in the PDF object issue here: #47

Maybe we could merge this, once Billingegroup/release-scripts#109 is merged all the way to v0.