Skip to content
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

Ue5 Initial Support via alonfaraj #5

Merged
merged 5 commits into from
Feb 6, 2023
Merged

Conversation

xxEoD2242
Copy link

@xxEoD2242 xxEoD2242 commented Aug 27, 2022

Mimics work done in Airsim: AirSim PR # 4672

Credit to @alonfaraj

Fixes: #

About

How Has This Been Tested?

Screenshots (if appropriate):

TODO:

  • Validate AirSim Multirotor pawn
  • Validate Multirotor API methods in Python
  • Validate vehicle API methods in Python
  • Validate multirotor API methods in C++
  • Validate vehicle API methods in C++
  • Validate utilizing PX4/Ardupilot
  • Validate keyboard controls with Vehicle

@noncomposmentis
Copy link

Is this only functional with the car APIs or did you just happen to not have to modify any of those files?

@xxEoD2242
Copy link
Author

Correct, only the car is supported. Work for Multirotor is ongoing

@alonfaraj
Copy link

Hey @noncomposmentis @xxEoD2242,
Actually multirotor works out of the box.
I have updated the original PR with an example video using the hello_drone.py.

@xxEoD2242
Copy link
Author

@noncomposmentis Awesome! Thank you!

I will reflect that in the PR. I still want to verify PX4/Ardupilot integration and a few other API calls to make sure but will not that as such.

@xxEoD2242
Copy link
Author

Leaving notes here:

UE5 requires a .NET Framework 3.0.X install, which can be done through VS 22 installer.

Still TBD on whether this works on Linux but issues with packaging are currently occurring.

@xxEoD2242
Copy link
Author

@alonfaraj Did you regenerate the animation .uasset file in the Unreal/Plugins/AirSim/Content folder? I tried to load the car and so did someone else is #17 and the animation files were corrupted since they are PhysX based.

@alonfaraj
Copy link

@xxEoD2242 Did you download the files from the link provided in the PR?
You should override the original files

@xxEoD2242 xxEoD2242 merged commit 55ce761 into CodexLabsLLC:main Feb 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants