| 04:37 | Dest123 | left the channel |
| 05:05 | Dest123 | joined the channel |
| 05:26 | Bertl_oO | off to bed now ... have a good one everyone!
|
| 05:26 | Bertl_oO | changed nick to: Bertl_zZ
|
| 05:42 | felix_ | left the channel |
| 05:42 | vup | left the channel |
| 05:43 | vup | joined the channel |
| 05:43 | felix_ | joined the channel |
| 05:44 | Bertl_zZ | left the channel |
| 05:44 | BAndiT1983 | left the channel |
| 05:44 | bluez_ | left the channel |
| 05:44 | eppisai_ | left the channel |
| 05:44 | felix_ | left the channel |
| 05:44 | vup | left the channel |
| 05:44 | Dest123 | left the channel |
| 05:45 | lexano | left the channel |
| 05:45 | mithro | left the channel |
| 05:45 | Spirit532 | left the channel |
| 05:45 | intrac | left the channel |
| 05:45 | balrog | left the channel |
| 05:45 | se6astian_ | left the channel |
| 05:45 | tpw_rules | left the channel |
| 05:45 | danieeel | left the channel |
| 05:45 | dos | left the channel |
| 05:45 | elafon | left the channel |
| 05:45 | illwieckz | left the channel |
| 05:45 | kbeckmann | left the channel |
| 05:45 | anuejn | left the channel |
| 05:45 | felix_ | joined the channel |
| 05:45 | vup | joined the channel |
| 05:45 | Dest123 | joined the channel |
| 05:45 | lexano | joined the channel |
| 05:45 | balrog | joined the channel |
| 05:45 | mithro | joined the channel |
| 05:45 | Spirit532 | joined the channel |
| 05:45 | elafon | joined the channel |
| 05:45 | Bertl_zZ | joined the channel |
| 05:45 | eppisai_ | joined the channel |
| 05:45 | bluez_ | joined the channel |
| 05:45 | BAndiT1983 | joined the channel |
| 05:45 | intrac | joined the channel |
| 05:45 | se6astian_ | joined the channel |
| 05:45 | illwieckz | joined the channel |
| 05:45 | tpw_rules | joined the channel |
| 05:45 | danieeel | joined the channel |
| 05:45 | kbeckmann | joined the channel |
| 05:45 | dos | joined the channel |
| 05:45 | anuejn | joined the channel |
| 05:50 | BAndiT1983 | left the channel |
| 05:50 | bluez_ | left the channel |
| 05:50 | eppisai_ | left the channel |
| 05:50 | BAndiT1983 | joined the channel |
| 05:50 | bluez_ | joined the channel |
| 05:50 | eppisai_ | joined the channel |
| 05:52 | Bertl_zZ | left the channel |
| 05:52 | Bertl_zZ_ | joined the channel |
| 12:12 | Dest321 | joined the channel |
| 12:15 | Dest123 | left the channel |
| 12:44 | Bertl_zZ_ | changed nick to: Bertl
|
| 12:45 | Bertl | morning folks!
|
| 13:36 | vnksnkr | is using the logic analyzer
| | 14:18 | vnksnkr | is turning off the analyzer
| | 16:40 | dos | changed nick to: dos1
|
| 16:44 | Dest321 | left the channel |
| 17:00 | Bertl | meeting time!
|
| 17:00 | tpw_rules | hello i'm here
|
| 17:00 | Bertl | please /msg me if you want to report ...
|
| 17:00 | eppisai_ | is here..
| | 17:01 | vup | is here
| | 17:01 | Bertl | vnksnkr: please go ahead with your report ...
|
| 17:01 | vnksnkr | Hi
|
| 17:02 | vnksnkr | So last week I've been working on the remote hardware
|
| 17:02 | vnksnkr | the issues with the new version of the remote have been solved
|
| 17:03 | vnksnkr | thanks to Bertl
|
| 17:03 | vnksnkr | this allowed me to test my gateware directly on the remote
|
| 17:04 | vnksnkr | and I've successfully been able to press buttons and dials on the remote from the Axiom Beta
|
| 17:05 | vnksnkr | to further inspect the signals being sent to the remote, Bertl hooked up the terminals on the remote to a logic analyzer
|
| 17:05 | Dest123 | joined the channel |
| 17:06 | vnksnkr | and as expected we were able to add bounces to the signals
|
| 17:06 | Bertl | excellent! got something to show us?
|
| 17:07 | vnksnkr | yeah sure, I've taken a few screenshots of the analyzer
|
| 17:07 | vnksnkr | https://usercontent.irccloud-cdn.com/file/rwaHBCch/Screenshot%20from%202021-08-16%2018-35-26.png
|
| 17:08 | vnksnkr | https://usercontent.irccloud-cdn.com/file/vocTOQco/Screenshot%20from%202021-08-16%2018-42-44.png
|
| 17:08 | Bertl | great!
|
| 17:08 | vnksnkr | the bounces are randomized , the first one simulates a push button and the second one simulates an encoder turn
|
| 17:09 | vnksnkr | https://usercontent.irccloud-cdn.com/file/rzG1HG1g/Screenshot%20from%202021-08-16%2014-09-26.png
|
| 17:09 | vnksnkr | these are the signals generated by an automated script, which would help in future testing on the remote
|
| 17:10 | vnksnkr | which brings me to the python framework
|
| 17:12 | vnksnkr | It can do basic operations like switch off/on bounces, load a seed value for the PRNG that generates bounces, and press buttons and turn dials with a specified time interval
|
| 17:12 | vnksnkr | https://github.com/vnksnkr/remote-test-system
|
| 17:13 | Bertl | looking forward to see that in future automated remote testing
|
| 17:13 | vnksnkr | this is the current status of the project, have added instructions on how to operate the remote test system
|
| 17:14 | vnksnkr | by this week I'll be submitting my final report
|
| 17:14 | Bertl | great! let us (mentors) know when you need a review
|
| 17:14 | vnksnkr | and add more documentation on the project
|
| 17:14 | vnksnkr | sure!
|
| 17:15 | vnksnkr | that's it from my side
|
| 17:15 | Bertl | thanks a bunch1
|
| 17:15 | Bertl | *bunch!
|
| 17:15 | Bertl | tpw_rules: please go ahead with your report
|
| 17:16 | tpw_rules | hello
|
| 17:16 | tpw_rules | last week I got the CMV12K sensor recording the test pattern properly to DRAM
|
| 17:16 | tpw_rules | i had to fix some more bugs in other parts of the gateware, but that is done. i have a couple more points to address in the review and then that will be ready to merge
|
| 17:17 | tpw_rules | hopefully i can get an implementation of the remapper done early this week because it's the last one...
|
| 17:17 | tpw_rules | but after that i will write up my report and submit it. i don't think it will be very long, basically just a list of PRs and some words on each
|
| 17:17 | Bertl | do you plan to continue contributing after GSoC?
|
| 17:17 | tpw_rules | i really want to try out the HDMI as that was part of the job, but that relies on Bertl getting a Beta ready with HDMI
|
| 17:18 | tpw_rules | it would be nice to finish off the job if that is okay
|
| 17:18 | tpw_rules | i.e. the remapper and HDMI output
|
| 17:18 | tpw_rules | so i guess it's okay if those aren't finished by the GSoC deadline?
|
| 17:18 | Bertl | ultimately this is up to your mentor, but I do not think it is a big problem
|
| 17:19 | tpw_rules | ok, well i can discuss it with them
|
| 17:19 | tpw_rules | that is it from me
|
| 17:19 | Bertl | if all goes well, we should have a working beta with HDMI out on Wednesday
|
| 17:19 | tpw_rules | ah okay
|
| 17:19 | vup | I think considering the difficulties with getting hardware up and running and some of the bugs in naps you progress is very good
|
| 17:20 | vup | And you are welcome to do contributions outside of GSoC
|
| 17:20 | tpw_rules | that's good to hear
|
| 17:20 | Bertl | but personally I would also suggest to focus on the final report now and not on the hardware verification
|
| 17:20 | tpw_rules | okay
|
| 17:21 | Bertl | because hardware is never without some issues
|
| 17:21 | tpw_rules | that is true :)
|
| 17:21 | Bertl | and the Beta - Remote connection showed us how long it can take for a seemingly simple setup to get it working
|
| 17:22 | tpw_rules | alright, well that is all from me
|
| 17:22 | vup | great progress as always!
|
| 17:23 | Bertl | thanks!
|
| 17:23 | Bertl | anybody else who would like to report but didn't /msg me yet?
|
| 17:23 | eppisai_ | Hi.. I would like to report next..
|
| 17:23 | Bertl | please go ahead then
|
| 17:24 | eppisai_ | Last week, after discussion with andrej, it was decided to use timer 1 instead of core timer..
|
| 17:25 | eppisai_ | *timer 2
|
| 17:25 | eppisai_ | So, have tried using 16 bit timer, to generate 1 sec interval interupt.. (changing PBCLK3 Freq)
|
| 17:26 | eppisai_ | Besides I am finishing up with all transition animations..
|
| 17:27 | eppisai_ | Because of hurdles and few problems during GSoC period , many things in my tasks are left..
|
| 17:29 | eppisai_ | So, will talk with mentors.... Want to continue working on the tasks..
|
| 17:29 | Bertl | I'm sure that won't be a problem
|
| 17:29 | eppisai_ | That's it from my side :)
|
| 17:29 | Bertl | thanks!
|
| 17:29 | Bertl | btw, now that we fixed the I2C for the EF Remote
|
| 17:30 | Bertl | it might be interesting to incorporate that into the Remote firmware and combine it with the Beta Control of the Remote Remote
|
| 17:31 | Bertl | but as I said previously, focus on the final report for now and getting the code cleaned up, etc
|
| 17:31 | Bertl | anybody else who would like to report?
|
| 17:32 | Bertl | I spent most of my spare time last week on getting the Remote and the Remote Control Shield working and tested
|
| 17:33 | Bertl | and this included fixing up the I2C code for the Remote to make it compatible with both Remote versions (v0.9 and v0.14)
|
| 17:33 | Bertl | we also discovered that the timing was a little too agressive, so when sending too many I2C messages, some of them didn't go through/come back correctly
|
| 17:34 | Bertl | which caused some confusion when we first tested (like ghost key events and such)
|
| 17:34 | Bertl | the firmware ICSP programmer was also updated to work properly with the EF Remote
|
| 17:35 | Bertl | and finally we did the Analyzer setup to allow vnksnkr to capture actual traces of the button/encoder bounces
|
| 17:36 | Bertl | not too much else happened, although there was some slow progess on getting the new Betas assembled and tested
|
| 17:36 | Bertl | that's it from my side for this week
|
| 17:36 | Bertl | if nobody else has anything to report, the meeting is concluded
|
| 17:37 | Bertl | thanks for your time and focus on the final report... have fund and good luck!
|
| 17:37 | vnksnkr | thank you :)
|
| 17:37 | tpw_rules | thank you
|
| 17:38 | eppisai_ | Thank you
|
| 17:39 | Bertl | my pleasure
|
| 17:39 | Bertl | off for now ... bbl
|
| 17:39 | Bertl | changed nick to: Bertl_oO
|
| 18:15 | Dest123 | left the channel |
| 18:21 | tpw_rules | anuejn or vup are you working on anuejn's beta
|
| 18:22 | Spirit532 | left the channel |
| 18:22 | vup | nope
|
| 18:22 | Spirit532 | joined the channel |
| 20:04 | vup | tpw_rules: I have been trying to track down your problem with the optimized version of the DramPacketRingbufferStreamWriter, but have been unable to yet
|
| 20:04 | vup | I tried to reproduce the problem with this: https://github.com/apertus-open-source-cinema/naps/pull/30
|
| 20:05 | vup | but that does not seem to work
|
| 20:05 | tpw_rules | vup: have you been able to demonstrate the brokenness yourself?
|
| 20:05 | vup | nope
|
| 20:06 | vup | (but I did not try with your pattern_dram_test yet
|
| 20:06 | tpw_rules | you should be able to see the difference between commit e97dc99 and 5fa5114 in my PR
|
| 20:07 | tpw_rules | alright, i was about to reimplement read_packet_to_file. should it just return a bytes object i guess?
|
| 20:08 | vup | yeah a bytes object seems fine
|
| 20:09 | vup | tpw_rules: I see that you inserted a StreamBuffer, but I cannot figure out why this should make a difference
|
| 20:09 | tpw_rules | i looked at the actual captured data and couldn't really see a cause
|
| 20:09 | tpw_rules | it just seemed random pieces were dropped on the floor
|
| 20:10 | vup | so you mean, the addresses of the transmitted data was right
|
| 20:10 | tpw_rules | yes
|
| 20:10 | vup | but sometimes there were chunks of the data that was missing?
|
| 20:10 | tpw_rules | well, largely
|
| 20:10 | tpw_rules | yeah
|
| 20:10 | tpw_rules | maybe the address didn't get incremented or something
|
| 20:10 | tpw_rules | but the first several kilobytes were fine
|
| 20:10 | tpw_rules | checks agian
| | 20:11 | vup | how often did you encounter that?
|
| 20:11 | tpw_rules | it's consistent
|
| 20:11 | tpw_rules | (iirc)
|
| 20:11 | vup | ok, but after how many bytes?
|
| 20:12 | tpw_rules | i'm checking
|
| 20:13 | tpw_rules | 0x84d80 bytes in is the first dropped line
|
| 20:15 | vup | hmm
|
| 20:21 | tpw_rules | look at /home/operator/tpw_rules/presumably_correct_pattern.bin for the result from 5fa5114 and /home/operator/tpw_rules/committed_broken_pattern.bin for the result from e97dc99
|
| 20:21 | tpw_rules | on the beta
|
| 20:40 | vup | now I just need to find something that can visually diff two 70Mb files :P
|
| 20:41 | tpw_rules | yeah, sorry. if you use `xxd -g 8 -c 48 <file> | cut -c 1-111` it comes out as one text line per camera two lines
|
| 20:41 | tpw_rules | well, per camera word.
|
| 20:41 | tpw_rules | hm, is there any problem with creating signals in elaborate?
|
| 20:41 | tpw_rules | i moved all the stuff in __init__ of DramPacketRingbufferCpuReader there and it works
|
| 20:44 | vup | no I think there used to be a problem with that, but no longer
|
| 20:44 | vup | emacs fares surprisingly well with diffing them
|
| 20:44 | tpw_rules | ok, i think i'm just going to do that. it's simpler than two separate loops
|
| 21:11 | Dest123 | joined the channel |
| 21:15 | vup | tpw_rules: interesting, so it looks like it is dropping 48 bytes there
|
| 21:15 | vup | maybe those actually get dropped because the address has now a 1 bigger latency?
|
| 21:16 | Dest123 | left the channel |
| 21:17 | tpw_rules | hm, i wonder if at that point the fifo fills up?
|
| 21:18 | vup | could be, thats now the point where all the `StreamInfo` blocks would be helpful :P
|
| 21:18 | tpw_rules | they are all still in the build
|
| 21:18 | tpw_rules | idk what they mean
|
| 21:20 | tpw_rules | btw also the SimpleStreamGearbox does not work. i assume all the tests pass but that results in 0 data transferred. maybe that is a clue. is it something on the transmit end in the Cmv12kRx?
|
| 21:20 | tpw_rules | the fifo should insulate that though....
|
| 21:20 | vup | hmm what does "does not work" mean?
|
| 21:21 | tpw_rules | like i replace StreamGearbox with SimpleStreamGearbox and everything compiles but instead of the whole image being transferred, 0 words are transferred
|
| 21:21 | vup | curious
|
| 21:21 | tpw_rules | i assume all the tests pass, they looked quite comprehensive
|
| 21:21 | vup | yeah the tests pass
|
| 21:21 | vup | can you upload a dump of `design` for a failing and a successful version with the DramPacketRingbufferStreamWriter thing?
|
| 21:22 | tpw_rules | sure, it will take a while though, i don't keep compiled gateware around
|
| 21:22 | vup | sure
|
| 21:22 | tpw_rules | (15-20 mins)
|
| 21:40 | tpw_rules | vup: https://pastebin.com/1zXbsHue https://pastebin.com/TuJmBppm
|
| 21:47 | vup | tpw_rules: hmm can you try to add a StreamInfo before and after the Gearbox in both cases?
|
| 21:47 | tpw_rules | those two commits?
|
| 21:47 | vup | yeah
|
| 21:48 | tpw_rules | ok
|
| 21:54 | tpw_rules | left the channel |
| 21:55 | tpw_rules | joined the channel |
| 22:04 | Dest123 | joined the channel |
| 22:22 | tpw_rules | vup: https://pastebin.com/x5UGgGUa https://pastebin.com/04KAxSU9
|
| 22:23 | tpw_rules | hm, looks like it is my fault. or at least whatever condition is propagating all the way back through the fifo and it is filling up
|
| 22:24 | vup | tpw_rules: I actually found a bug with the SimpleGearbox: https://github.com/apertus-open-source-cinema/naps/commit/5f5c9897f19c63f3d85ab028910864b6286521f6
|
| 22:26 | tpw_rules | ah, does it not handle non power of 2 division ratios?
|
| 22:26 | vup | it *did* not
|
| 22:26 | vup | yeah
|
| 22:28 | tpw_rules | ok, let me rebase and see what difference that makes
|
| 22:35 | vup | tpw_rules: yeah, having some valid_not_ready > 0 for a stream that cannot actually buffer the data means you will drop some, which matches exactly what you can observe
|
| 22:36 | tpw_rules | isn't the StreamBuffer supposed to buffer one thing?
|
| 22:37 | tpw_rules | and why is the FIFO backing up? there must be some interaction with the fix that breaks bursting or adds excessive delays
|
| 22:37 | vup | yeah the StreamBuffer is basically a one deep fifo
|
| 22:37 | vup | I think somewhere you just need to be able to buffer one more word
|
| 22:37 | tpw_rules | but the fifo is 2048 words deep
|
| 22:37 | tpw_rules | it should not fill up
|
| 22:41 | tpw_rules | like i said i think there's some new problem in the dram writer where it is spending too much time waiting for some reason
|
| 22:41 | vup | yeah maybe
|
| 22:41 | vup | I am looking into that right now
|
| 22:44 | tpw_rules | ok, so SimpleStreamGearbox works now and significantly simplifies the logic but the timing is of course much worse
|
| 22:44 | tpw_rules | >_<
|
| 22:45 | vup | lol
|
| 22:45 | tpw_rules | 9000 -> 7000 nets
|
| 22:46 | anuejn | loooool
|
| 22:49 | tpw_rules | all the timing issues are in the DRAM domain anyway
|
| 22:50 | tpw_rules | hm, yeah it is better in the sensor domain. maybe i will switch later
|