search

fselius / qtcopter

2 stars 0 forks source link

Connect PX4Flow sensor #39

Open fselius opened 9 years ago

fselius commented 9 years ago
noamyogev84 commented 9 years ago

Just added a service to the px4flow script (flow.py) so we can set new waypoints during flight. Thanks.

@Vasily

roeemz commented 9 years ago

@noamyogev84, are you resetting the PID integral part upon doing so?

efiShtain commented 9 years ago

there is the PID start/stop service, I can reset all on a stop request, so if new point is set, someone would have to stop and restart the PID and it will reset the integral part other option is to add specific service for resetting the integral part but i don't see how it helps cause last known error has to be reset as well

2015-08-16 1:00 GMT+03:00 roeemz notifications@github.com:

@noamyogev84 https://github.com/noamyogev84, are you resetting the PID integral part upon doing so?

— Reply to this email directly or view it on GitHub https://github.com/fselius/qtcopter/issues/39#issuecomment-131455885.

Vasily commented 9 years ago

It's only changing the flow state (current deltas)

noamyogev84 commented 9 years ago

I think i forgot. Will fix this.

On Sunday, August 16, 2015, roeemz notifications@github.com wrote:

@noamyogev84 https://github.com/noamyogev84, are you resetting the PID integral part upon doing so?

— Reply to this email directly or view it on GitHub https://github.com/fselius/qtcopter/issues/39#issuecomment-131455885.

fselius commented 9 years ago

@serbuh Do we have a permanent place for the flow sensor? Best if it's aligned with the center of gravity Z axis.