acmpesuecc / acmpesuecc/traffic_simulation

Data Structure Alternatives - Commenting on efficiency

Aperta
#8 46 commenti 0 reazioni 1 assegnatario Rivendicata da @sameermanvi Vedi su GitHub
BOUNTY:75 hacknight-2025 hacktoberfest
Lingua principale
Python
Stelle
1
Fork
13
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

# Bounty Points: 75

## Branch Instructions
push changes to a new *'ds'* branch

***

**Explanation:**

Current clock list management is **inefficient**
Removes items while iterating: `for i in clock: if i[2]==t: clock.remove(i)`
$O(n^2)$ complexity due to `list.remove()` in loop
No priority queue for event scheduling

Current problematic code:
```python
for i in clock:
if i[2] == t:
clock.remove(i) # $O(n)$ operation in $O(n)$ loop
```
### Possible fix/approach:

* Replace list with **`heapq`**-based **priority queue**
* Use event-based scheduling with `(time, event\_data)`
* Alternative: **dictionary** with time as key
* Benchmark before/after improvements
* Document complexity improvements

---

### Maintainer notes:

* Current approach causes **performance issues** with large graphs
* Focus on **clock/event management data structure**
* Should maintain same simulation logic

---

### Resources:

* Python `heapq` documentation
* Event-driven simulation patterns

---

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.