-
Notifications
You must be signed in to change notification settings - Fork 217
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add TMap implementation to lua #755
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code looks good to me.
I haven't tested it.
Status: We've had several people successfully test this PR in the discord server. |
There may be some issues that we need to clear up. |
Such as ? |
When pushing keys/values as so: pa.PoseContainer.Poses[1].TrackToBufferIndex = {[1] = 11, [2] = 22, [3] = 33} The values get pushed to the wrong keys. |
Description
Adds
TMap
support to lua.Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Tested by running a sample project in UE 5.3.2 and calling every function added.
Checklist
Please delete options that are not relevant. Update the list as the PR progresses.