Elias Hogstvedt

Ah that makes sense. I hope you find the time to do it. :) here's the manually fixed lua file if you want to diff it https://dl.dropboxusercontent.com/u/244444/ammo.lua

I believe this is ready for a review. I haven't tested it that much apart from the test suite and there are some unresolved type errors left. Would be nice...

> @CapsAdmin could you delete [8092507](https://github.com/CapsAdmin/pac3/commit/80925072db392d97600649fd272d3bf5c0b1e2a9) from master? > > I wanted it deleted a while ago because it botched the commit history. Seems like it might've caused some other...

As mentioned further up, they have been mostly moved. What features specifically do you miss and why? I assume you meant those you underlined, but can you explain further?

The internals of how bone names work is stupid. It should have been using the actual bone name when serializing and then the friendly names as translation. Because right now...

This is sort of expected. There might be a solution for this but generally when an entity is not visible it should not render the pac outfit. The intended workaround...

If I understand this, I think this is expected behavior. You need to check duplicate on the group for it to work.

Yes something is definetly wrong here, I'll have a look at it soon. I think while I'm at it, I'll make it so duplicate applies on all entities that match,...

I like this idea. I can use `ply:GetNWVarTable()` to populate available properties and while i'm at it `ply:GetSaveTable()` with https://wiki.facepunch.com/gmod/Entity:GetInternalVariable

You could try turning off backups maybe? add `do return end` at line 221 in your screenshot and see if it happens then