This is actively occurring in Arsenal, causing the game to become more or less unplayable. This error is thrown whenever GetPlayerFromCharacter is called without a Model instance.
This occurs a lot when instance class checking isn’t provided (because previously this would simply return nil), which is pretty much every occurrence of this function being used ever. An extremely critical bug which really shouldn’t have occurred in the first place.
My sentry logs for my game have suddenly been throwing server and client errors. This is throughout older servers and new ones. This is definitely game.Players:GetPlayerFromCharacter() function which throws the same thing: Argument 1 is invalid: expected 'Model' instance type. I’m seeing this in studio as well – earlier today this problem did not exist.
Can confirm, this is also affecting Football Fusion and a variety of other games. The game is expecting a model as others have stated & returning nil error.
This change has been reverted. Game servers should resume the old behavior in the next few minutes; clients will have to be restarted for the revert to take effect. We’ll investigate how to roll this change out without causing this.
Just for clarification, do you mean this change (the method throwing the error) was an unintended side effect of a different change, or that it was a premature release of an upcoming intended breaking change in this method?
TL;DR should we expect this error to return in the future?
The method throwing an error is an unexpected effect of a change to provide more accurate type information in reflection. This error, or any others like it, should not recur going forward.