Skip to content

Conversation

@vishal332008
Copy link
Member

No description provided.

SenjuZoro and others added 30 commits April 12, 2025 14:37
New firespark added! 🚒
fixed default settings and config.json
…ronPowerup.py

This pull request resolves two critical issues in `elPatronPowerup.py`:

---

🔧 Issue 1: Bomb initialization failed
- ❌ Error:
  TypeError: Bomb.__init__() takes 1 positional argument but 8 were given
- 🔍 Cause:
  self._pm_old_bomb(...) was passing multiple positional arguments instead of keyword arguments.
- ✅ Fix:
  Replaced with keyword-based argument passing:
  self._pm_old_bomb(
      position=position,
      velocity=velocity,
      bomb_type=new_bomb_type,
      blast_radius=blast_radius,
      bomb_scale=bomb_scale,
      source_player=source_player,
      owner=owner
  )

---

🎨 Issue 2: animate_array() received too many arguments
- ❌ Error:
  TypeError: animate_array() takes 4 positional arguments but 5 were given
- 🔍 Cause:
  An unnecessary True was passed as a fifth argument.
- ✅ Fix:
  Removed the extra argument:
  bs.animate_array(self.texts[type], 'color', 3, {
      0: (1, 0, 0),
      0.2: (1, 0.5, 0),
      0.4: (1, 1, 0),
      0.6: (0, 1, 0),
      0.8: (0, 1, 1),
      1.0: (1, 0, 1),
      1.2: (1, 0, 0)
  })

---
Fix: Correct Bomb init arguments and animate_array parameter in elPat…
update readme to only fetch latest changes
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.

7 participants