Skip to content

Update to API 8#1

Open
vishal332008 wants to merge 136 commits intobombsquad-community:public-serverfrom
imayushsaini:public-server
Open

Update to API 8#1
vishal332008 wants to merge 136 commits intobombsquad-community:public-serverfrom
imayushsaini:public-server

Conversation

@vishal332008
Copy link
Member

No description provided.

imayushsaini and others added 30 commits May 4, 2025 21:54
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
syncing ballistica 1.7.50
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