3d inspection fix

[hbsmith]

Currently 3D hover inspection is broken.

Minimal non working example:

using Agents, Plots, Random, GLMakie

@agent struct Dummy(ContinuousAgent{3,Float64}) 
     speed::Float64
end

function setup(;
    n_agents = 100,
    extent = (100, 100, 100),
)
    space3d = ContinuousSpace(extent)
    model = StandardABM(Dummy, space3d; agent_step! = agent_step,
                        container = Vector, scheduler = Schedulers.Randomly(), rng = Xoshiro(314))
    for _ in 1:n_agents
        add_agent!(model,(1.,1.,1.),1.)
    end
    return model
end

function agent_step(dummy, model)
    move_agent!(dummy, model, dummy.speed)
end

model=setup()

fig, abmobs = abmexploration(model; agent_size=3)

fig

Note that there's a visible arrow but none of the default inspection information.

I don't have a great understanding of Makie, but it seems like the issue had something to do with the way the position was being translated, and thus never recognizing any Agents in the position of the mouse. The changes I made fix this as minimally as I was able to, such that it avoids Bounds errors, and displays information of multiple agents if they're in the same position.

With the fix:

If someone thinks there's a better way to do this, be my guest, but at least it works nows.

Note that this should be merged after #1159

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.27%. Comparing base (8b5b456) to head (ed567a0).
Report is 187 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #1162       +/-   ##
===========================================
+ Coverage   70.12%   85.27%   +15.14%     
===========================================
  Files          42       38        -4     
  Lines        2718     2614      -104     
===========================================
+ Hits         1906     2229      +323     
+ Misses        812      385      -427     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[ameligrana]

I'm wondering: isn't idx already agent_idx? we should really try to avoid collect(allids(model))

[ameligrana]

Nice work by the way :-)

[Datseris]

@hbsmith thank you for this, can you please address the comment by @Tortar so that we can merge this in?