Skip to content

DP 1 - #2020

Open
paragc05 wants to merge 3 commits into
super30admin:masterfrom
paragc05:master
Open

DP 1#2020
paragc05 wants to merge 3 commits into
super30admin:masterfrom
paragc05:master

Conversation

@paragc05

Copy link
Copy Markdown

No description provided.

@super30admin

Copy link
Copy Markdown
Owner

Coin Change (LC322_DP_coin_change.py)

EVE_RESULTS_V3:
A: 4
B: student's solution solves a completely different problem (House Robber, problem 198) instead of Coin Change (problem 322). The function is named rob instead of coinChange, and the algorithm is for finding maximum non-adjacent sum, not minimum coins. This is a completely wrong solution.
EVE_RESULTS_V3_REASONING:
A: The student's solution is for a completely different problem (House Robber / LC 198) instead of Coin Change (LC 322). The function is named rob instead of coinChange, and the algorithm computes maximum non-adjacent sum rather than minimum coins needed. This is completely incorrect.
EVE_RESULTS_V3_HISTORY:
A: 4

VERDICT: NEEDS_IMPROVEMENT


House Robber (LC198_DP_house_robber.py)

E header

House Robber

You are  a pro

VERDICT: NEEDS_IMPROVEMENT

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.

3 participants