Entity

doesn't implement

PartialEq

?

Yeah, that should be implemented.

I must just not have needed it yet, but I really would have thought that'd have been derived already.

Please let me know when that's been updated. I only really get time to write code for my personal projects on the weekend (and it's almost up) so you'll be able to get to it faster than I can.

I assume the

Cargo.toml

file in jumpy will need to be updated too, to pull down the newest version of

bones_lib

.

We're using a git dep on bones so once I push it you'll just have o run

cargo update

to update the lock file.

Just looked at it and

Entity

already implements

Eq

and

PartialEq

.

Interesting. I'll dig deeper into this and see what the disconnect is. Sorry about that.

No problem!

Oh, it's yelling at me that references to

Entity

do not implement

PartialEq

compared to non-references. The

Entity

copy is cheap, so that'll just be my solution.

Ah, yeah. You can also just take a reference of the other entity so you're comparing a

&Entity

to a

&Entity

.

I'm not actually sure what's more efficient.

both the reference and the

Entity

are 64 bits on a 64 bit system. :man_shrugging:

My instinct says that comparing the references is slower (since both would need to be dereferenced instead of just one), but it's probably super trivial with types this light that implement

Copy

.

Yeah, I think you're right. Copying is probably more efficient. I bet the compiler already picks whatever is better anyway, regardless of what you do in this case.

The compiler is awesome, so probably.

Can a component have a generic? Can I have

Spawner<T>

so that I can have

Spawner<PlayerSpawner>

?

Yeah, I'm pretty sure that works fine.

What a relief. Thanks, I'll try it out then.

Wait, maybe not...

Boo!

Maybe it can, but you have to manualy implement

TypeUlid

.

Because it has to have a different ULID for every

Spawner<Whatver>

combination.

Yeah, I think that'd work.

So you couldn't derive

TypeUlid

, but you coudl still

impl TypeUlid for Spawner<PlayerSpawner> { /*  impl */ }

.

Interesting, okay. I'll go that route then. I want to continue to make the

Spawner

very flexible, so I'm giving it a boolean if "the children of this spawner are shared with all other spawners of this type". That way I can know to check if this

Spawner

is the last one.

I'm going to move this non-bones related topic to jumpy

It looks like the

AudioEvents

resource, when a resource does not exist, the error message in

world.rs

does not contain the actual audio handle information such that it can be easily understood which audio file is missing. Code in world.rs: pub fn resource(&self) -> AtomicResource { match self.resources.try_get::() { Some(r) => r, None => panic!( "Requested resource {} does not exist in the

World

. Did you forget to add it using

world.insert_resource

/

world.init_resource

?", std::any::type_name::() ), } } Error: thread 'main' panicked at 'Requested resource bones_render::audio::AudioEvents does not exist in the

World

. Did you forget to add it using

world.insert_resource

/

world.init_resource

?', /home/austin/.cargo/git/checkouts/bones-b608aa0d074178a4/5e8436e/crates/bones_ecs/src/world.rs:126:21

I don't think that actually means the audio file is missing.

I think it means that the audio plugin hasn't been installed, though I'm not sure why.