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

Camera replaced by Cinematographic Camera (CinemAirSim) #3779

Closed
wants to merge 6 commits into from
Closed

Camera replaced by Cinematographic Camera (CinemAirSim) #3779

wants to merge 6 commits into from

Conversation

ppueyor
Copy link
Contributor

@ppueyor ppueyor commented Jun 9, 2021

Fixes: #
-Integrates the CineCameraComponent into AirSim

About

An AddOn for AirSim that includes all the tools to integrate a cinematographic camera and the needed tools to control/access to it.
More info available in repo: https://github.com/ppueyor/CinemAirSim
Or conference paper: https://arxiv.org/abs/2003.07664

How Has This Been Tested?

Ubuntu 20 and UE4 4.26
Different demos and API calls
Results available in repo

Screenshots:

https://www.youtube.com/watch?v=OJGNaitWZVA

@ppueyor
Copy link
Contributor Author

ppueyor commented Jun 21, 2021

Hi, the checks need approval from the maintainers. Hope the last commit does the trick and passes the checks.

@ppueyor
Copy link
Contributor Author

ppueyor commented Jul 12, 2021

Hi, I fixed the last conflicts of this PR, but I still need a maintainer to check if it builds correctly.

Thanks!

@jonyMarino
Copy link
Collaborator

/azp run microsoft.AirSim

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jonyMarino
Copy link
Collaborator

Hi @pablo! Your PR seems to be failing at the block's packaging stage

@ppueyor
Copy link
Contributor Author

ppueyor commented Jul 15, 2021

Hi Jonathan! Thanks for your answer. It builds in the rest of the checks, right?

I use it in Ubuntu, play it in the blocks package and runs perfectly.
I don't know what happens.
How can I fix this issue?

Thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants