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

ENABLING HIL MODE IN JSBSIM BRIDGE #27

Open
xfahad100 opened this issue Oct 22, 2020 · 29 comments · May be fixed by #28
Open

ENABLING HIL MODE IN JSBSIM BRIDGE #27

xfahad100 opened this issue Oct 22, 2020 · 29 comments · May be fixed by #28
Labels
HITL question Further information is requested

Comments

@xfahad100
Copy link

it seems like it is only sitl enabled. I have successfully enabled serial by changing mavlink tcp port to ttyACM0 in the configs file.
I want to enable HIL mode and serial communication at the same time. a little help would be appreciated. thanks

@Jaeyoung-Lim Jaeyoung-Lim linked a pull request Oct 22, 2020 that will close this issue
@Jaeyoung-Lim
Copy link
Contributor

@xfahad100 I created a draft pull request from the WIP branch I was working on in #28

Please feel free to move it forward

@Jaeyoung-Lim Jaeyoung-Lim added HITL question Further information is requested labels Oct 22, 2020
@xfahad100
Copy link
Author

serial connectivity is resolved, but for hitl simulation, connectivity of gcs udp port is also required.

@xfahad100
Copy link
Author

hil mode is achieved. px4 is sending data through mavlink to qgc in hil simulation

@xfahad100
Copy link
Author

I will close this issue in 24 hours, thanks for your time.

@Jaeyoung-Lim
Copy link
Contributor

@xfahad100 Awesome, so the PR just worked out of the box for you?

@xfahad100
Copy link
Author

@Jaeyoung-Lim No , I had to make some changes related to connectivity with Qgroundcontrol.

@xfahad100
Copy link
Author

They were very minor changes . all credit goes to you! @Jaeyoung-Lim

@Jaeyoung-Lim
Copy link
Contributor

@xfahad100 Would you mind making a PR with your changes included?

@xfahad100
Copy link
Author

@Jaeyoung-Lim Yes I will in few days! I just need to comment outunnecessary changes.

@Jaeyoung-Lim
Copy link
Contributor

Jaeyoung-Lim commented Oct 24, 2020

@xfahad100 Awesome, thanks! I will keep this open until it is fully resolved and merged 😄

Also, just pushing the unclean version of it would still help since it will save time on debugging the issue with the original PR

@xfahad100
Copy link
Author

#35 #36 merge them @Jaeyoung-Lim

@Jaeyoung-Lim
Copy link
Contributor

@xfahad100 Is there a reason those two are separated in two PRs?

@xfahad100
Copy link
Author

@Jaeyoung-Lim uhh no,, I'm a newbie on github and I cant figure out how to compare changes that I've made in 2 files. So there are 2 prs for each file ..... sorry for the mess

@Jaeyoung-Lim
Copy link
Contributor

@xfahad100 I tried your PR, but was not able to get GPS position. Were you able to fly with the changes you made?

@xfahad100
Copy link
Author

ACM0 is workin fine but USB0(telemetry) is giving tx overflow error.

@Jaeyoung-Lim
Copy link
Contributor

ACM0 is workin fine

Can you confirm that you can fly with the vehicle in the simulation? which airframe were you using?

@Jaeyoung-Lim
Copy link
Contributor

@xfahad100 Any updates?

@xfahad100
Copy link
Author

It starts flying by itself in HIL mode and does not respond to px4 data....

@Jaeyoung-Lim
Copy link
Contributor

Jaeyoung-Lim commented Jan 27, 2021

@xfahad100 Are you saying that it didn't work for you from the start?

@xfahad100
Copy link
Author

@Jaeyoung-Lim is mag sensor plugin not working at the moment?
I will share my results once I am able to fly my vehicle.

@Jaeyoung-Lim
Copy link
Contributor

@xfahad100 Now I am a bit confused. You are saying it worked before and not anymore?

@xfahad100
Copy link
Author

@Jaeyoung-Lim bruhh I never said that.. I am now able to get gps location but I am not able to fly ..

@Jaeyoung-Lim
Copy link
Contributor

Jaeyoung-Lim commented Feb 10, 2021

@xfahad100 Yes exactly. I have asked you multiple times if it was working since October last year, but you never answered. So are you saying it has never worked since october?

@xfahad100
Copy link
Author

I'm working on it n its almost done...

@Jaeyoung-Lim
Copy link
Contributor

@xfahad100 I am assuming that means it never worked for you.

@xfahad100
Copy link
Author

@Jaeyoung-Lim it is working for me right now

@xfahad100
Copy link
Author

At least it takes input from rc and throw it out to jsbsim

@xfahad100
Copy link
Author

Bro I am doing hitl successfully with px4 and jsbsim and also implementing autopilot

@xfahad100
Copy link
Author

But I have excluded qgroundcontrol

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
HITL question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants