Clarify documentation of update remove_empties.R

[matanhakim]

Fixes #568

Feel free to edit or change, it is a bit difficult to write a clear description.
Added a clarification for lower and higher values.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (3769ab0) to head (f89767b).

❗ Current head f89767b differs from pull request most recent head be57f3b

Please upload reports for the commit be57f3b to get more accurate results.

Additional details and impacted files

@@            Coverage Diff            @@
##              main      #569   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           27        27           
  Lines         1274      1274           
=========================================
  Hits          1274      1274           

Files	Coverage Δ
R/remove_empties.R	100.00% <ø> (ø)

[billdenney]

The update looks good. Can you please update the full documentation (devtools::document()) so that everything is consistent and push that update?

[matanhakim]

Hi,
I added the documentation.
You're welcome to have a look and provide feedback as necessary.
Thanks!

[billdenney]

This looks good to me!

[sfirke]

This is a tricky one to describe in a brief and clear way 😅 @matanhakim and @billdenney what do you think of my attempt that built on Matan's - anything still to improve?

[billdenney]

It looks good to me with the current edits.

[matanhakim]

This looks great to me. Indeed a tricky explanation!