Make Store.items a dequeue about concurrentsim.jl HOT 9 CLOSED

The problem with Deque is that it does not allow to delete an arbitrary element.
I have used a Store only with mutable objects. For immutable type we have that a==b is identical to a===b.
One way I can solve this, is boxing immutable types...

from concurrentsim.jl.

Boxing immutable objects or using a Dict with as key the object and as value the number of times it is stored. Both are doable. I have a slight preference for the latter. All input is welcome!

from concurrentsim.jl.

I had some spare time;) straightforward implementation of a Store based on a Dict is in the master branch.

julia> using SimJulia

julia> env = Simulation()
Simulation time: 0.0 active_process: nothing

julia> store = Store{Symbol}(env)
Store{Symbol}

julia> put(store, :apple)
SimJulia.Put 1

julia> put(store, :apple)
SimJulia.Put 2

julia> put(store, :pear)
SimJulia.Put 3

julia> store.items
Dict{Symbol, UInt64} with 2 entries:
  :pear  => 0x0000000000000001
  :apple => 0x0000000000000002

What do you think?

from concurrentsim.jl.

I agree! Closed

from concurrentsim.jl.

Sorry for the late answer ... SimJulia had a low priority but it will be better in the future;)
A Dequeue is more logical. I will implement it.

from concurrentsim.jl.

What is the path forward here?

from concurrentsim.jl.

I definitely vote for the latter as well. I'm not a fan of boxing and using a dictionary would make it clear how many items you have.

from concurrentsim.jl.

Looks great. I've started using this already. Unless you think it needs further work, I think we can close this issue.

from concurrentsim.jl.

I've been using this new implementation and it works great for my purposes. Unless @riccardosven suggests otherwise, we should close this issue.

from concurrentsim.jl.