changed create agents in tutorial

[DarshPareek]

Summary

This PR changes the tutorials to make them consistent with Mesa 3.1

Motive

Make Mesa tutorial consistent with 3.1 release

Implementation

• Changed intro_tutorial.ipynb
• Changed MoneyModel.py

Usage Examples

class MoneyModel(mesa.Model):
    """A model with some number of agents."""

    def __init__(self, n=10, seed=None):
        """Initialize a MoneyModel instance.

        Args:
            N: The number of agents.
            width: width of the grid.
            height: Height of the grid.
        """
        super().__init__(seed=seed)
        self.num_agents = n
        

        # Create agents
        MoneyAgent.create_agents(self,n)

[github-actions]

Performance benchmarks:

Model	Size	Init time [95% CI]	Run time [95% CI]
BoltzmannWealth	small	🔵 +0.8% [+0.2%, +1.3%]	🔵 +0.2% [+0.0%, +0.3%]
BoltzmannWealth	large	🔵 +1.2% [+0.8%, +1.5%]	🔵 +0.3% [-0.3%, +0.9%]
Schelling	small	🔵 +0.8% [+0.5%, +1.1%]	🔵 +0.3% [+0.1%, +0.4%]
Schelling	large	🔵 +0.9% [+0.5%, +1.3%]	🔵 +3.6% [+2.5%, +4.8%]
WolfSheep	small	🔵 +3.2% [+2.8%, +3.5%]	🔵 +0.9% [+0.7%, +1.0%]
WolfSheep	large	🔴 +3.7% [+3.5%, +4.0%]	🔴 +7.0% [+5.5%, +8.5%]
BoidFlockers	small	🔴 +8.0% [+7.5%, +8.6%]	🔵 +0.6% [-0.3%, +1.4%]
BoidFlockers	large	🔴 +7.4% [+6.7%, +8.2%]	🔵 +0.6% [-0.0%, +1.2%]

[EwoutH]

Thanks for the PR!

[EwoutH]

I did assign @ashish-nagmoti to issue #2537 by the way. Next time please do check if an issue is assigned, and if it is, ask first.

@ashish-nagmoti would you like to review this PR instead?

[DarshPareek]

In the latest update I have changed the arguments to be more explicit in create_agents function and added

x = self.rng.integers(0, self.grid.width, size=(n,)) 
y = self.rng.integers(0, self.grid.height, size=(n,))

to make positioning of agents better

[EwoutH]

I’m good, but let’s see if Jan also is.

[quaquel]

I am fine with this, although I personally would prefer to move self.grid.place_agent(a, (i, j)) into the MoneyAgent.

[EwoutH]

Personally I won't bother with updating the old spaces, we will replace them soon anyways.

@DarshPareek Could you make your PR title and description a bit more clear/descriptive?