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.

Sign up for GitHub

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

Jump to bottom

Deduplicate - window sliding #208

Open
hobovsky opened this issue Oct 21, 2023 · 2 comments
Open

Deduplicate - window sliding #208

hobovsky opened this issue Oct 21, 2023 · 2 comments
Labels
discussion/deduplicate Discussion and vote what to do with duplicate kata

Comments

@hobovsky
Copy link
Contributor

  1. Squeaky Window
  • running max of windows
  • 7kyu
  • Satisfaction 88% with 310 completions
  • 1 pending issue
  • 5 languages + 1 pending translation
  • Published Sep 2015, author active
  1. Moving Average
  • 7kyu
  • Satisfaction 93% with 700+ completions
  • running average of windows
  • No pending issues
  • python only + pending JS translation
  • Published Feb 2019, author inactive

Both kata require return null/None on invalid configuration of inputs: null array, window length less than 1 or larger than len(arr).
@hobovsky hobovsky converted this from a draft issue Oct 21, 2023
@hobovsky hobovsky moved this to Discussing in Deduplication process Oct 21, 2023
@hobovsky hobovsky added the discussion/deduplicate Discussion and vote what to do with duplicate kata label Oct 21, 2023
@hobovsky
Copy link
Contributor Author

I think that there's not much difference between max of a window and avg of a window, and 1. can be kept as more complete, and 2. is not needed.

@EloiseRosen
Copy link

Vote for keep 1 only.

@ejini6969 ejini6969 mentioned this issue Jan 14, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
discussion/deduplicate Discussion and vote what to do with duplicate kata
Projects
Deduplication process
Status: Discussing
Milestone
No milestone
Development

No branches or pull requests
2 participants
@EloiseRosen @hobovsky