betaflight / betaflight-tx-lua-scripts

Collection of scripts to configure Betaflight from your TX (currently only supported in OpenTx)
GNU General Public License v3.0
590 stars 142 forks source link

Position OSD Elements via Script #463

Closed druckgott closed 1 year ago

druckgott commented 1 year ago

I added a new page where all osd elements are listet and you can select via the first filed by changing the number up and down each osd element. On the lower area you can see the name of the osd element and more lower you can see the position value and the profil where this osd element is visible.

You can change the position and also the profil and then save the value and the osd will be updated.

druckgott commented 1 year ago

I did something wrong I think, I want to remove all commits expect the last two how descriped here: https://github.com/betaflight/betaflight-tx-lua-scripts/pull/457#issuecomment-1312770596 But anyting go wrong. I change from the 9 commits listed only the 3 one I do not want to have to fixup and then I pushed again over cmd git: git push --force-with-lease But then I see also old comits inside?! Do I have to change the git rebase -i HEAD~9 to git rebase -i HEAD~5 and then change pick to fixup only for the one I want to hold anymore?

haslinghuis commented 1 year ago

Please check: https://github.com/betaflight/betaflight/wiki/Betaflight-Contribution#squash-your-commits

klutvott123 commented 1 year ago

I'm trying this now, and I'm not sure I understand how it's supposed to work. I can't change anything. It goes through all retries and the changes won't take effect.

druckgott commented 1 year ago

The Adress you can change to select the osd element you want to change. The you have to go down to the line where the on off points are if all 3 points are off you will not see the element in the osd If one of it is one you can see the element in the osd, this the first number you can change the position, if you are if you then save the osd element should be on a mother position if you atere in the right osd page. So my idea was at first only to move a element if you see on the field it's not fitting perfekt lx you like it and you have no pc here.

druckgott commented 1 year ago

Here is a Video https://youtu.be/4ZpBPOYd2qk

druckgott commented 1 year ago

I'm trying this now, and I'm not sure I understand how it's supposed to work. I can't change anything. It goes through all retries and the changes won't take effect.

Did you see the video? What system do you have for me it works perfekt, I have a taranis x light pro.

klutvott123 commented 1 year ago

Yes, I saw the video. I have a taranis qx7. Tested with fport and tracer. FC config is minimal. Just rx and osd enabled. I can make changes in the configurator and they show up in the script. It may be that my test setup is a little too minimal.

druckgott commented 1 year ago

What I'm not sure if the addresses are always the same or if they depend what's aktivated for example gps, but I do not know how to read out the availible elements due to the config like the betaflight editor do. If anybody can help me there maybe that's better?!

klutvott123 commented 1 year ago

What I'm not sure if the addresses are always the same or if they depend what's aktivated for example gps, but I do not know how to read out the availible elements due to the config like the betaflight editor do. If anybody can help me there maybe that's better?!

They should be the same. I got it working here by clearing the values table before preparing the payload when saving. See one of the comments above for explanation.

klutvott123 commented 1 year ago

I'll just add that this doesn't work with ELRS because MSP_OSD_CONFIG results in a huge payload spread across more than 2 CRSF frames. Not much we can do about it here as it's a ELRS limitation.

druckgott commented 1 year ago

I did most changes expect this two:

druckgott commented 1 year ago

I changed all the things, hopefully I get everything ;-). Thx for your good Ideas it helps me to improve my programming skills ;-).

klutvott123 commented 1 year ago

Almost there. Just some minor things that need fixing. There are probably more optimizations that could be done here with regard to memory use, but it's ok for now I think.

druckgott commented 1 year ago

hopefully all is ok now ;-)

klutvott123 commented 1 year ago

Looks good 🙂 Just need to squash/fixup to one commit now

druckgott commented 1 year ago

Hopefully all is fine now, I´m not shure if I did everything right, but I hope only on commit is visible with the right input.