- Posts: 4402
- Forum
- News, Announcements and Feedback
- Feedback & Questions
- PB, request for code review and merge back(2)
PB, request for code review and merge back(2)
- PhracturedBlue
- Offline
bitbucket.org/PhracturedBlue/deviation/i...atus=new&status=open
that are related to the Devo10 and see which can be closed? I think most/all of them.
Please Log in or Create an account to join the conversation.
- suvsuv
- Topic Author
- Offline
- Posts: 268
Yes, I am keeping track of bugs related to devo10. For those bugs that I replied with fixed, please have them closed as I don't have privilege to do so.PhracturedBlue wrote: Also, can you go through the issues here:
bitbucket.org/PhracturedBlue/deviation/i...atus=new&status=open
that are related to the Devo10 and see which can be closed? I think most/all of them.
BTW, will you replace the 12narrow font with 12normal font for devo8?
Please Log in or Create an account to join the conversation.
- PhracturedBlue
- Offline
- Posts: 4402
I'm not planning on it. We only have enough space on the Tx for 1 full-unicode font. that is 15normal for devo8 and 12normal for devo10. all other fonts need to be a subset (generally just pure ascii)suvsuv wrote: BTW, will you replace the 12narrow font with 12normal font for devo8?
Please Log in or Create an account to join the conversation.
- suvsuv
- Topic Author
- Offline
- Posts: 268
2) Fix a major bug in calibration
Please Log in or Create an account to join the conversation.
- PhracturedBlue
- Offline
- Posts: 4402
Please Log in or Create an account to join the conversation.
- suvsuv
- Topic Author
- Offline
- Posts: 268
I don't get your point. When adding a feature , must I create a bug on the bitbucket?PhracturedBlue wrote: Auto-dimmer is not on bitbucket, so I can't review
Please Log in or Create an account to join the conversation.
- FDR
- Offline
Please Log in or Create an account to join the conversation.
- suvsuv
- Topic Author
- Offline
- Posts: 268
Sorry, my network's issue and I don't realize it was broken in the middle of pushing. Just pushed codes in.FDR wrote: I think he meant you haven't pushed it yet...
Please Log in or Create an account to join the conversation.
- PhracturedBlue
- Offline
- Posts: 4402
2 things:
1) why do you need to check it in the GUI? I don't think that is the right place to check the Dimmer. in the main loop (for powerdown) and in the button and touch handlers (for power up) should be sufficient, it seems to me.
2) I don't see any value in your low-priority functions. The screen update at 100msec is low speed enough for anything we need in my opinion, so just put it there.
Please Log in or Create an account to join the conversation.
- suvsuv
- Topic Author
- Offline
- Posts: 268
Changed as per your suggestionPhracturedBlue wrote: Overall, the autodim feature looks ok.
2 things:
1) why do you need to check it in the GUI? I don't think that is the right place to check the Dimmer. in the main loop (for powerdown) and in the button and touch handlers (for power up) should be sufficient, it seems to me.
I don't see the need to check back-light as quick as once per 100msec. once per seconde is sufficient for this low priority task. But this is not a blocking issue for both you and me, so you can just move the backlight checking into the redraw loop in your repo.PhracturedBlue wrote: 2) I don't see any value in your low-priority functions. The screen update at 100msec is low speed enough for anything we need in my opinion, so just put it there.
Today I make quite a few checkin, please take a look and have them merged back to your repo:
1) Merged your latest update except change related Expo&DR in the mixer setup
2) Provide configuration UI for auto-dimmer setting in both devo10 and devo8, please verify it in your devo8 TX
2) In the Expo&Dr setting page, you moved 2 graphes from mixer setup into its subpage. This is not a good solution, compared with my current one. What I do is to have them switched to the active viewpoint based on what left item is selected.
2) Fix improper font setting for the timer in the main page when time is up. The config.ini under filesystem/devo10/media must be merged back
3) Fix bug about drawing BMP in devo10
Please Log in or Create an account to join the conversation.
- PhracturedBlue
- Offline
- Posts: 4402
I don't understand. As far as I can tell, the current code works exactly like the previous code, except that you now have the vertical bar as well. you created 3 xy graphs, but there seems to be no way to view more than one at a time, and the way to switch them is to select the switch in question, which redraws the graphs from scratch. What did I miss? Is there another way to switch the current XY graph on the Expo page?suvsuv wrote: 2) In the Expo&Dr setting page, you moved 2 graphes from mixer setup into its subpage. This is not a good solution, compared with my current one. What I do is to have them switched to the active viewpoint based on what left item is selected.
Please Log in or Create an account to join the conversation.
- suvsuv
- Topic Author
- Offline
- Posts: 268
of entering and exiting subpages from time to time
Please Log in or Create an account to join the conversation.
- FDR
- Offline
Jeez! I didn't realized, that the two other curve options are there.
In general clicking on such a source or switch button means inverting it, so I haven't even tried.
I thought that page is still incomplete...
Please Log in or Create an account to join the conversation.
- PhracturedBlue
- Offline
- Posts: 4402
Please Log in or Create an account to join the conversation.
- FDR
- Offline
FDR wrote: The Expo&DR page could be arranged like the Timers are. There the two timer is on two separate scrolled pages whether they take all the place or not. In this way the DR page could be split to three parts with one (optional) graph per page, but since it has more options to set, it should have two pages per graph. The only thing is that those pages should have some label indicating which part of the whole you actually see.
Please Log in or Create an account to join the conversation.
- suvsuv
- Topic Author
- Offline
- Posts: 268
No matter how we do ,the deviation is not a easy-to-use system for new users. A manual is needed instead of putting tutorial items.PhracturedBlue wrote: I didn't realize it worked that way. It is certainly not inuitive. maybe at least show some text indicating what is being displayed? It would be somewhat more intuitive, and would make it more clear what was going on.
That is what I am going to do. But man, you should give me some patient as I already drive out this system to devo10 very quick. Currently, I only slow down a little bit and focus on bug-fixing in order to let the deviation10 becomes a stable build.PhracturedBlue wrote: Also, I'm not yet convinced of using sub-pages for the expo-d/R. it makes configuration very slow. I think I'd rather just have a couple more text-selects on the main page.
Please Log in or Create an account to join the conversation.
- PhracturedBlue
- Offline
- Posts: 4402
Take no offense. The work you've done has been great and is certainly appreciated by me and many others.suvsuv wrote: But man, you should give me some patient as I already drive out this system to devo10 very quick. Currently, I only slow down a little bit and focus on bug-fixing in order to let the deviation10 becomes a stable build.
We are now close enough to doing a proper release that I want to make sure the interface is in its final state (for this release). Before release, I need to write the documentation, and it takes me a long time to do documentation. If the interface changes, I need to redo it, so having the interface stable is important to the final steps.
Do you have any plans to have a customizeable main page like we do for the devo8? We're obviously more limited in space, but (for instance) being able to configure whether the model image is shown, or that space supports text boxes or toggles would probably be very helpful. Please let me know what your plans are in this regard.
Please Log in or Create an account to join the conversation.
- FDR
- Offline
I just was really surprised, that the funcionality is there at all.
The point is, that PB created that page in the devo 8 to simplify defining the usual flight mode dependent settings on one page, i.e. without the need of the complex mixer. If it's not on one page, it's not that important, I think.
IMO the DEVO10 could simply have only the simple and the complex mixers... (But don't delete it, for heaven's sake! )
Please Log in or Create an account to join the conversation.
- PhracturedBlue
- Offline
- Posts: 4402
It is probably true, but would make sharing models between the devo8 and devo10 more challenging since the template is stored in the model file.FDR wrote: IMO the DEVO10 could simply have only the simple and the complex mixers... (But don't delete it, for heaven's sake! )
Please Log in or Create an account to join the conversation.
- suvsuv
- Topic Author
- Offline
- Posts: 268
So far, the deviation10 is proved to be quite stable from what I heard from about 5 test users.PhracturedBlue wrote:
We are now close enough to doing a proper release that I want to make sure the interface is in its final state (for this release). Before release, I need to write the documentation, and it takes me a long time to do documentation. If the interface changes, I need to redo it, so having the interface stable is important to the final steps.
Do you have any plans to have a customizeable main page like we do for the devo8? We're obviously more limited in space, but (for instance) being able to configure whether the model image is shown, or that space supports text boxes or toggles would probably be very helpful. Please let me know what your plans are in this regard.
And most of the bugs that we knew have been fixed.
In the weekend, I will improve the Expo&DR page to draw all items into the same page.
In terms of the customizeable main page, could you give me some suggestion about what you want to configureļ¼ then I will get it done this weekend also. Of coz, please take the LCD resolution constraint into account
Please Log in or Create an account to join the conversation.
- Forum
- News, Announcements and Feedback
- Feedback & Questions
- PB, request for code review and merge back(2)
- Home
- Forum
- News, Announcements and Feedback
- Feedback & Questions
- PB, request for code review and merge back(2)