JuliaDynamics/Agents.jl

Move `add/remove_agent_from_space!` to the public API

Open

#1 154 ouverte le 8 avr. 2025

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)Julia (146 forks)batch import
good first issuespace interaction api

Métriques du dépôt

Stars
 (904 stars)
Métriques de merge PR
 (Merge moyen 12h 3m) (1 PR mergée en 30 j)

Description

Following from the discussion here: https://discourse.julialang.org/t/remove-agent-from-space-but-not-scheduler/127842/10

The summary is: there are good reasons for users to be able to directly add or remove agents to a space, without removing them from existence (the ABM itself) alltogether.

To do this we need to make the add/remove_agent_from_space! public, and also:

  • In its docstring document that this function is already called internally in add_agent! or remove_agent! and therefore does nto have to be manually called by the user if they are doing this operation.
  • Test EXTENSIVELY in ALL SPACES whether using these functions directly without add/remove_agent! leads to any breakages.

This feature is a pre-requisitive step for #910 .

Guide contributeur

Direction de recherche
Examinez les implémentations actuelles de add agent from space! et remove agent from space!; assurez vous qu'elles fonctionnent correctement dans tous les types d'espace; ajoutez de la documentation et des tests.
Stack technique
Aucun
Domaine
api
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Julia
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65