03:14 | comradekingu | left the channel | |
03:14 | comradekingu | joined the channel | |
04:37 | Bertl | off to bed now ... have a good one everyone!
| |
04:37 | Bertl | changed nick to: Bertl_zZ
| |
06:12 | BAndiT1983|away | changed nick to: BAndiT1983
| |
08:10 | se6ast1an | good day!
| |
08:48 | futarisIRCcloud | left the channel | |
08:51 | mumptai | joined the channel | |
10:56 | futarisIRCcloud | joined the channel | |
12:06 | Bertl_zZ | changed nick to: Bertl
| |
12:06 | Bertl | morning folks!
| |
12:47 | se6ast1an | good day
| |
14:26 | metal_dent[m] | se6ast1an: for the 2nd step of the ImageButton to add text along the image, where exactly will the text be? under the image or by the side?
| |
14:27 | BAndiT1983 | hi metal_dent[m], would say bot variants are valid
| |
14:28 | se6ast1an | you could start with icon left, text right - thats the classic layout I would say
| |
14:29 | metal_dent[m] | we'll need different constructors for these layouts..?
| |
14:29 | BAndiT1983 | ?
| |
14:29 | BAndiT1983 | what about an enum?
| |
14:29 | BAndiT1983 | and setter, or extend constructor with proper argument, maybe also with default value
| |
14:30 | BAndiT1983 | like ImageButton::Icon
| |
14:30 | BAndiT1983 | or ImageButton::IconAndText
| |
14:30 | BAndiT1983 | and maybe the setter for position, like SetTextPosition(ButtonTextPosition::Bottom)
| |
14:33 | metal_dent[m] | okay, I'll try extending the constructor
| |
16:23 | BAndiT1983 | metal_dent[m]: regarding bootloader, this is what the current version should output -> https://pastebin.com/raw/uvkELx0t
| |
16:23 | BAndiT1983 | had to fix my board, the resistor is very tiny
| |
16:30 | Bertl | display is flashing again
| |
16:31 | Bertl | so I would be surprised if the PIC32 was working correctly
| |
16:32 | metal_dent[m] | yes, I was trying to flash ':D
| |
16:36 | BAndiT1983 | display flashing is normal
| |
16:36 | BAndiT1983 | 500ms on, 500ms off
| |
16:38 | Bertl | no, actually it is more like a defective screen in a movie
| |
16:38 | Bertl | flickering
| |
16:39 | BAndiT1983 | oh, that's not what BL should do
| |
16:40 | metal_dent[m] | Bertl: can you see the ACM0 file in /dev ?
| |
16:40 | BAndiT1983 | mine is sitting to my right on the table and blinking by using the backlight LED, also USB-UART and LVP works fine, we should investigate what the difference is
| |
16:41 | Bertl | metal_dent[m]: there is none
| |
16:41 | BAndiT1983 | and this is strange, same code, but i have /dev/ttyACM0
| |
16:42 | BAndiT1983 | can try to flash BL from the CI
| |
16:42 | Bertl | as I said, it doesn't look healthy
| |
16:42 | Bertl | so I wouldn't bother with USB for now, I would try to figure out what goes wrong with the PIC startup
| |
16:43 | Bertl | probably the bootloader jumps into areas which are not initialized which leads to random code execution or similar
| |
16:44 | metal_dent[m] | > metal_dent: there is none
| |
16:44 | metal_dent[m] | i just flashed BL and tried to checked minicom but that hung so i force closed it maybe that's why :/
| |
16:44 | BAndiT1983 | which BL have you used?
| |
16:44 | metal_dent[m] | *check
| |
16:44 | BAndiT1983 | CI build is not flashable for me, tried IPE
| |
16:44 | BAndiT1983 | but my local one does
| |
16:45 | metal_dent[m] | i tried from the repo
| |
16:45 | BAndiT1983 | but note that my local is equal to the repo version, haven't touched it for long time
| |
16:45 | BAndiT1983 | please try to build locally and upload, have to check why CircleCI build is not working
| |
16:46 | BAndiT1983 | local build 170kb and working correct, but CI build is 184kb and failing
| |
16:47 | BAndiT1983 | metal_dent[m]: please try to build locally, how big is your .hex file?
| |
16:47 | BAndiT1983 | will check CI script in the meantime
| |
16:48 | Bertl | more flickering
| |
16:48 | BAndiT1983 | can i send you my file?
| |
16:48 | mumptai | left the channel | |
16:49 | Bertl | it's probably simpler to directly upload it
| |
16:49 | metal_dent[m] | > metal_dent: please try to build locally, how big is your .hex file?
| |
16:49 | metal_dent[m] | 180K
| |
16:49 | BAndiT1983 | ah, right, will upload
| |
16:49 | mumptai | joined the channel | |
16:50 | BAndiT1983 | https://gofile.io/d/OEjI82
| |
16:50 | Bertl | metal_dent[m]: it's in /tmp
| |
16:51 | metal_dent[m] | usb+ ?
| |
16:51 | Bertl | OEjI82.hex
| |
16:52 | Bertl | ah sorry, that is a link where you can download stuff
| |
16:52 | Bertl | (not the file itself)
| |
16:52 | Bertl | AXIOM_Remote_Bootloader.hex
| |
16:54 | Bertl | that looks a lot better
| |
16:54 | Bertl | now the display is blinking
| |
16:55 | BAndiT1983 | then please try minicom and reset the board
| |
16:55 | BAndiT1983 | if not possible to reset, then reflash it while minicom is open
| |
16:55 | BAndiT1983 | you should see the content of KMW (as far as i remember it was the default one in the LVP module)
| |
16:56 | metal_dent[m] | but for minicom we'll need ACM0...?
| |
16:56 | Bertl | which is now there
| |
16:56 | Bertl | (after the correct firmware was loaded)
| |
16:57 | metal_dent[m] | BAndiT1983: yes, now I can see the contents in minicom
| |
16:59 | BAndiT1983 | ok, that's very good, now we need to investigate why the circleci build is failing, luckily i can execute their docker script locally, maybe then i will find a mistake
| |
18:34 | Spirit532 | left the channel | |
18:34 | Spirit532 | joined the channel | |
18:44 | BAndiT1983 | changed nick to: BAndiT1983|away
| |
18:50 | metal_dent[m] | BAndiT1983: how's this -> https://pasteboard.co/J97QZWU.png
| |
19:13 | se6ast1an | black :) -> much better than random!
| |
19:13 | se6ast1an | and the icon and text also looks good, how do you determine the placement currently?
| |
19:16 | metal_dent[m] | as we only need the text and image in a button in top or bottom bars so _width > _height so in Draw() i added this condition and for them x will be +_width/4 for image and +3*_width/4 for text
| |
19:18 | se6ast1an | I would recommend an alternative approach: in the end we want the group of image and text to be centered in the button
| |
19:19 | se6ast1an | so you could introduce a new member variable (with setter and getter) called margin
| |
19:19 | se6ast1an | it defines the width of the gap between image and the start of he text
| |
19:20 | se6ast1an | to center the group you take image_width + gap + textwidth together
| |
19:20 | metal_dent[m] | yeah, that sounds better
| |
19:33 | metal_dent[m] | the "gap" should be a fixed value, right?
| |
19:44 | se6ast1an | member variable with setter and getter
| |
19:51 | metal_dent[m] | but won't the gap between the image and text same in all cases? we want to center "image_width + gap + text_width"
| |
20:02 | se6ast1an | not necessarily
| |
20:13 | BAndiT1983|away | changed nick to: BAndiT1983
| |
22:00 | se6ast1an | heading off to bed for today, good night
| |
22:13 | BAndiT1983 | changed nick to: BAndiT1983|away
| |
22:27 | BAndiT1983|away | changed nick to: BAndiT1983
| |
23:18 | mumptai | left the channel | |
23:51 | BAndiT1983 | changed nick to: BAndiT1983|away
|