Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Some comments regarding documentation #1

Open
bkamins opened this issue Jul 15, 2023 · 5 comments
Open

Some comments regarding documentation #1

bkamins opened this issue Jul 15, 2023 · 5 comments

Comments

@bkamins
Copy link

bkamins commented Jul 15, 2023

  1. It would be super useful if each of the options and their combinations were presented by example (i.e. to have one data set and on it present all functions with all options enabled/disabled) - I know this is work in progress, but it is really important to have :)
  2. In documentation of running it would be good to add a x-link to tapering definition.
  3. In documentation of running it is not 100% clear what padding kwarg does. Reading the docs now it would suggest that it will replace NaN in the output if it is present. But I am not sure if it is the intended meaning of the description.
  4. In documentation of datastream it is not clear if you write vector you mean AbstractVector and when you write matrix you mean AbstractMatrix.
  5. With weights and running when tapering happens it is not 100% clear how weights are processed. Are weights re-weighted to add up to 1 (or slightly less than 1) or not
  6. In "When a window is provided weights, is important to normalize those weights." it is not clear if this is a general info, or a requirement of the package. The reason is that many functions in general do not require weights to be normalized, however, maybe this package has some special implementations for some functions that assume they are normalized.
@JeffreySarnoff
Copy link
Owner

Thank you. This is on point!

@JeffreySarnoff
Copy link
Owner

remade docs to be intelligible :)

< now to use the docs as specs and provide fuller conformance testing >

@bkamins
Copy link
Author

bkamins commented Jul 25, 2023

The https://jeffreysarnoff.github.io/WindowedFunctions.jl/dev/#Introducing-Windowed-Functions is awesome.

BTW: you might want to have a look at duckdblabs/db-benchmark#9 and we could discuss how to best add Julia to these benchmarks.

@JeffreySarnoff
Copy link
Owner

JeffreySarnoff commented Jul 25, 2023 via email

@bkamins
Copy link
Author

bkamins commented Jul 25, 2023

Excellent. Thank you!
BTW. if you are at JuliaCon2023 we could work on it there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants