From 248faa3cdf9f250a863a47de8bd8babbec7cb112 Mon Sep 17 00:00:00 2001 From: Vijay Janapa Reddi Date: Thu, 18 Sep 2025 14:01:51 -0400 Subject: [PATCH] Simplify implementation pattern - remove heavy performance analysis - Removed formal PERFORMANCE NOTE section (too academic) - Integrated performance tips into HINTS when relevant - Keep focus on practical implementation guidance - Less intimidating for students while still teaching good practices - Performance considerations only when they really matter --- .claude/agents/module-developer.md | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/.claude/agents/module-developer.md b/.claude/agents/module-developer.md index 8e8a8a1c..dcd6725c 100644 --- a/.claude/agents/module-developer.md +++ b/.claude/agents/module-developer.md @@ -124,13 +124,8 @@ def method_name(self, params): HINTS: # <- OUTSIDE solution block! - Use np.function() for [specific reason why] - Remember to handle [edge case] because [why it matters] - - Consider memory: [what to think about for efficiency] - Check shape compatibility to avoid [common error] - - PERFORMANCE NOTE: # <- OUTSIDE solution block! - - Time complexity: O(?) - think about [what drives complexity] - - Space complexity: O(?) - consider [what uses memory] - - Bottleneck: [Common performance issue to avoid] + - Performance tip: [only if relevant - e.g., "avoid creating copies"] """ ### BEGIN SOLUTION # Complete implementation with comments explaining key decisions