{"action":"created","issue":{"url":"https://api.github.com/repos/betaflight/betaflight/issues/1898","repository_url":"https://api.github.com/repos/betaflight/betaflight","labels_url":"https://api.github.com/repos/betaflight/betaflight/issues/1898/labels{/name}","comments_url":"https://api.github.com/repos/betaflight/betaflight/issues/1898/comments","events_url":"https://api.github.com/repos/betaflight/betaflight/issues/1898/events","html_url":"https://github.com/betaflight/betaflight/issues/1898","id":197694078,"number":1898,"title":"Battery monitoring throttle reduction","user":{"login":"Yamaford","id":13566044,"avatar_url":"https://avatars1.githubusercontent.com/u/13566044?v=4","gravatar_id":"","url":"https://api.github.com/users/Yamaford","html_url":"https://github.com/Yamaford","followers_url":"https://api.github.com/users/Yamaford/followers","following_url":"https://api.github.com/users/Yamaford/following{/other_user}","gists_url":"https://api.github.com/users/Yamaford/gists{/gist_id}","starred_url":"https://api.github.com/users/Yamaford/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Yamaford/subscriptions","organizations_url":"https://api.github.com/users/Yamaford/orgs","repos_url":"https://api.github.com/users/Yamaford/repos","events_url":"https://api.github.com/users/Yamaford/events{/privacy}","received_events_url":"https://api.github.com/users/Yamaford/received_events","type":"User","site_admin":false},"labels":[{"id":390830578,"url":"https://api.github.com/repos/betaflight/betaflight/labels/Feature%20Request","name":"Feature Request","color":"fbca04","default":false},{"id":330919548,"url":"https://api.github.com/repos/betaflight/betaflight/labels/For%20Discussion","name":"For Discussion","color":"bfe5bf","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":37,"created_at":"2016-12-27T12:08:33Z","updated_at":"2018-03-25T08:30:09Z","closed_at":null,"author_association":"NONE","body":"With brushed microquads becoming more popular and no osd or telemetry, would it be possible to use the battery monitoring function to say cut the throttle in half(3/4?) at a certain voltage? This would give a physical indicator of a low battery and maybe prevent damaged lipo or un controllable decent. If the throttle is cut but you still have control you can land. "},"comment":{"url":"https://api.github.com/repos/betaflight/betaflight/issues/comments/375953996","html_url":"https://github.com/betaflight/betaflight/issues/1898#issuecomment-375953996","issue_url":"https://api.github.com/repos/betaflight/betaflight/issues/1898","id":375953996,"user":{"login":"nmw01223","id":34967821,"avatar_url":"https://avatars3.githubusercontent.com/u/34967821?v=4","gravatar_id":"","url":"https://api.github.com/users/nmw01223","html_url":"https://github.com/nmw01223","followers_url":"https://api.github.com/users/nmw01223/followers","following_url":"https://api.github.com/users/nmw01223/following{/other_user}","gists_url":"https://api.github.com/users/nmw01223/gists{/gist_id}","starred_url":"https://api.github.com/users/nmw01223/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nmw01223/subscriptions","organizations_url":"https://api.github.com/users/nmw01223/orgs","repos_url":"https://api.github.com/users/nmw01223/repos","events_url":"https://api.github.com/users/nmw01223/events{/privacy}","received_events_url":"https://api.github.com/users/nmw01223/received_events","type":"User","site_admin":false},"created_at":"2018-03-25T08:30:08Z","updated_at":"2018-03-25T08:30:08Z","author_association":"NONE","body":"Interesting. I have just set up an environment under Linux (Ubuntu 14.04 \nLTS). Cloned the repository and had a quick look at the code. Built some \ntargets (Omnibus, OmnibusF4, SPRacing) OK but not yet tested anything \nwhile I wait for a new FC (Omnibus F3) to arrive to practice on (don't \nwant to trash an existing one).\n\nCompletely unfamiliar with the code, but looks to me that adjustments \ncould be placed in and around fc_rc.c#310-325, updating \nrcCommand[THROTTLE] by picking up the average cell voltage from \ngetBatteryAverageCellVoltage() at battery.c#473 (ThrottleToUse = \nThrottleInput * (NominalCellVoltage / CellVoltage) with a few range \nchecks and limits around it).\n\nNeed to add a few NV parameters though - nominal cell voltage, max \nvoltage adjustment  limit, min voltage adjustment limit maybe. Haven't \nfigured out yet how to add new parameters to the NV CLI list, though \nlooking at contents of settings.c and pg.c.\n\nIf anyone could point me to any documentation about the overall \nstructure of the betaflight software, it would save a lot of time ... ?\n\nOn 25/03/18 06:55, bnmoore wrote:\n>\n> @Engineereric <https://github.com/Engineereric> @AndrewFarley \n> <https://github.com/AndrewFarley> @nmw01223 <https://github.com/nmw01223>\n> I landed here in search of the same. Got a build environment working \n> (Visual Studio Code) and submitted a pull request.\n> #5528 <https://github.com/betaflight/betaflight/pull/5528>\n>\n> —\n> You are receiving this because you were mentioned.\n> Reply to this email directly, view it on GitHub \n> <https://github.com/betaflight/betaflight/issues/1898#issuecomment-375947529>, \n> or mute the thread \n> <https://github.com/notifications/unsubscribe-auth/AhWRDaORFJ1j1J_fsoJMrJM0QBlC7X9-ks5thzFSgaJpZM4LWO-e>.\n>\n\n"}}
{"actor":{"display_login":"nmw01223"}}