So how easy is it to get things changed for that page. I don’t recommend that we use it because there are a lot of parameters that changed and may not be set correctly from that page. We need to add the 5 pt throttle curve and the governor settings
So arming in auto is not allowed for Heli’s and Multi’s. As long as I can remember, neither frame could arm in auto.
It is only for the landing detector to disarm. Once it touches the ground and disarms it relaxes the pitch to full bottom with the rotor still spinning at rated speed (although slowing down now after disarm).
The reason for it was people failing to set it, and failing to set it properly. We had one very bad explosion because of it where the helicopter rolled over because it blew the EKF from trying to pull too much negative pitch and never disarmed. It blew the pitch links off, the helicopter launched like a rocket and mowed a 30 foot tree down. If somebody would’ve been in the way their life would’ve been discontinued.
I actually removed it from the ArduHeli builds over a year ago. I finally PR’d the removal to master after it was discussed on the Dev Call as being a safety issue.
You said that dynamic flight code is still active. I wasn’t sure if that had been removed at the same time as Land_Col_Min.
Yes, correct sorry, got the name wrong. Actually, I thought maybe you would go the other way. It’s currently 0-1000. Others have been rescaled 0-100. Seems this one could be as well. Actually this one probably needs less precision than RSC_Setpoint would, so it’s a good candidate for sure.
Not quite, if I understand what you’re asking. Tridge can make these… not sure what they’re called, classes of parameters that don’t even show up in the list until you enable the master. Such as PLND_ENABLED.
Just some of them could be moved. The less commonly used ones, like H_Gov params. It’s a way to reduce the apparent number of parameters for beginners. Just a suggesting.
As I scroll through the massive number of params, I wish it was more common. Such as MNT, LGR, CAM, etc.
Shouldn’t be too bad. You just have to work with Michael Oborne. Before he was full time it took a bit to get his attention, but I think he’s working on MP full time now?
I was talking with Tridge about it. Yeah so Multis can’t either. I think Randy is against it. I was too. But my opinions on this stuff have changed.
Sorry, what do you mean here? When I created it, Land_Col_Min was active the entire time when winding up and down. You never got full negative collective until you were flying Dynamically.
Sorry, I can’t agree with that logic. There are so many things that people can set improperly, that will result in crashes. You just can’t get rid of them all.
ie: somebody could set H_Col_Mid too low. Same effect. Set H_Col_Mid too high, and it will take off into your face uncontrollably. Set ATC_RATE_PITCH_P to be negative, or reverse the AHRS orientation and it will fly backwards at you.
Install a DDFP tail system, but don’t set the parameter. While disarmed on the ground, if you yaw left, then right, it will arm then spin up. I’ve done that one myself!
As it is now with Col_Mid, you will increase the instances of people having mid-air disarms, and it will crash onto their heads. Believe me, it happened to me twice when I set Land_Col_Min too high (ie: not enough negative). With Col_Mid, it will happen even more often. Believe me. I’ve already had this happen once in the sim just playing with it for a couple hours.
Again, you can’t just delete features because somebody got it wrong. There will be nothing left of the code. We had a user with a massive vibration problem on the run-up. MP was screaming at them. They took off anyway. Then they did disarm gesture in the air, heli spun to the right while descending, impacted and exploded.
What feature do you delete to solve that?
It’s a shame to limit functionality for advanced users because some make mistakes.
Despite the fact it has been flown and tested by many users without a single problem, if you don’t like it revert the commit. It’s only a couple lines of code for each frame type and re-instate the param in MotorsHeli.
Hang on, the original function was also flown and tested by many users without a problem. My point is just that removing it solves nothing. Get a parameter wrong, and you get the exact same effect.
1 Like
I already posted a solution for you in post 25.
The commit is here if you need a guide to revert it. It’s a very simple change.
When you revert it, please provide clear, concise documentation on how to set your param, and make sure to address the issues with pulling negative pitch on the ground with three or more blade non-rigid rotors with droop stops and coning hinges. That was never done and there is now some people flying those, although they are flying my code and not ArduPilot.
Your documentation should note that both COL_MID and COL_MIN must be set to zero thrust pitch with fully-articulated rotors, and LAND_COL must be set to zero.
Clear, concise documentation on HOW to set it was never provided in the past. All it said was:
Minimum collective position in PWM microseconds while landed or landing
What does that mean to the user? PWM microseconds, let’s see - geeze my collective is 1000 to 2000 pwm settings. This setting is 0-500? WTF?
Very poorly designed and documented system throwing in different scaling factors that the end user does not understand.
@Rob_Lefebvre I am OK with putting that capability back into the code. But I just had a thought of how to make it easier for set up and require less inputs initially unless the user really needs the capability. Why not make a new parameter that allows the user to set the landing collective but make it relative to the mid collective setting. It would be defaulted to zero so that the mid collective position would be used and if non-zero would indicate a percent from the mid position to the min position. This way users would not have to set this parameter if they don’t need negative collective and would prevent users from getting into ground resonance issues as they would if the land collective min parameter was not set properly. Thoughts?
Yeah, that makes total sense. Nice compromise.
So would I want the parameter to be a negative percentage to indicate that it’s below mid? Or keep it as a positive percentage described as the percentage below mid?
Not sure. Could go wrong either way if somebody doesn’t read the instructions and sets it wrong.
Seems the convention is we almost never use negative numbers.
Well, just program your transmitter accordingly. I think that to say that it’s holding helicopters back is a bit extreme. Also wont work on gas helis.
1 Like
Actually that highlights precisely the point. To not have a transmitter at all.
And there’s no reason that this could not be made to work on gas helis.
What can be made to work, vs what’s safe and practical is two different things. There’s a reason some people can get 500 hrs on their piston engine, and others only get 50 hours before it seizes up. In cold weather I run at high idle, clutch engaged and rotors turning for 2-3 minutes sometimes for warmup. In hot weather I’ll run it for the same time at the same speed for cooldown so when I shut it down the temperature doesn’t spike and stick piston rings. Which will score a cylinder and seize it up.
Turbines are even more important for proper warmup and cooldown if you don’t want a $2,000 engine overhaul to due to cracked combustors, or letting an autopilot over-temp it by pushing the ITT over 1000C on spoolup.
Lack of autonomy is not holding helicopters back. Aviation authorities are. Those authorities that do the certification of the aircraft are not impressed with full autonomous features. They are impressed by a Pilot in Command retaining 100% control of the helicopter in all phases of the flight. That’s why drones are so tightly regulated commercially. They don’t trust them because they’re too automatic. That’s why a drone can’t be hired out to DOT for a survey of a U.S. or State highway, but a manned aircraft can be.
I don’t really understand the relevance of this.
A) Your usage of a gas heli, should not preclude somebody else who is flying, say a 249g helicopter in full auto mode.
B) You’ve presented no problems with a programmable system could not be made to handle. Those sorts of situations are, frankly, trivial to solve in software. And in fact, would be more reliable than an operator being left to handle it manually. You even said it yourself, you do it right, and others don’t. I would have loved to have sold a gas powered helicopter, but I won’t go there until all of it can be automated. Right down to oil and fuel injection.
Ardupilot is nowhere near stable enough to be flown safely without a transmitter. Lets hit that stage first, eh?
There are lots of people doing that already. Just not with Heli.
Nobody is going to force you to use a feature you don’t want to use.
But if you want to help make whatever improvements you think are needed, please contribute.
There might be people doing that, but as long as EKF lane changes are a thing, I would not call it safe.
So maybe keep focusing on features that make ardupilot safe, rather than features for toys.
2 Likes