Skip to content

solution#5

Open
ellepannitto wants to merge 1 commit intoASPP:masterfrom
ellepannitto:local_maxima
Open

solution#5
ellepannitto wants to merge 1 commit intoASPP:masterfrom
ellepannitto:local_maxima

Conversation

@ellepannitto
Copy link

We changed the tests! (not to cheat, it's a real discussion)

@pberkes
Copy link
Contributor

pberkes commented Sep 7, 2022

I think it would fail this test:

def test_local_maxima_not_a_plateau():
    values = [1, 2, 2, 3, 1]
    expected = [3]
    maxima = local_maxima(values)
    assert maxima == expected

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

Successfully merging this pull request may close these issues.

2 participants