Current Server Time: 23:01 (Central Europe)

#apertus IRC Channel Logs

2019/06/20

Timezone: UTC


00:15
rektide_
joined the channel
00:19
rektide_
was looking at recorder/monitors for my camera & was thinking how much i wanted open source hardware to be able to do SDI, so i showed up here to just say hi & bow before how much awesome work ya'll have done
00:21
Bertl
you're very welcome!
00:39
rektide_
changed nick to: rektide
01:28
BAndiT1983
changed nick to: BAndiT1983|away
03:53
Bertl
off to bed now ... have a good one everyone!
03:53
Bertl
changed nick to: Bertl_zZ
04:19
illwieckz_
joined the channel
04:22
illwieckz
left the channel
06:31
niemand
joined the channel
06:31
niemand
left the channel
06:31
niemand
joined the channel
06:32
sebix
joined the channel
06:49
niemand
left the channel
07:02
sebix
left the channel
07:18
BAndiT1983|away
changed nick to: BAndiT1983
07:58
illwieckz_
left the channel
08:06
se6astian|away
changed nick to: se6astian
09:48
BAndiT1983
changed nick to: BAndiT1983|away
09:56
Umori_
left the channel
09:59
Umori
joined the channel
10:34
Umori
left the channel
10:38
Umori
joined the channel
11:19
se6astian
changed nick to: se6astian|away
12:54
Bertl_zZ
changed nick to: Bertl
12:54
Bertl
morning folks!
12:57
BAndiT1983|away
changed nick to: BAndiT1983
13:36
sebix
joined the channel
13:57
se6astian|away
changed nick to: se6astian
14:01
BAndiT1983
changed nick to: BAndiT1983|away
14:17
RexOrCine|away
changed nick to: RexOrCine
14:36
sebix
left the channel
14:48
se6astian
changed nick to: se6astian|away
14:55
se6astian|away
changed nick to: se6astian
14:55
se6astian
left the channel
14:55
se6astian|away
joined the channel
14:56
se6astian|away
changed nick to: se6astian
14:56
se6astian
left the channel
14:56
RexOrCine
left the channel
14:56
Nira|away
left the channel
14:56
BAndiT1983|away
left the channel
14:56
philippej
left the channel
15:00
Nira|away
joined the channel
15:00
BAndiT1983|away
joined the channel
15:00
BAndiT1983|away
changed nick to: BAndiT1983
15:00
philippej|away
joined the channel
15:00
philippej|away
changed nick to: philippej
15:00
RexOrCine|away
joined the channel
15:00
se6astian
joined the channel
15:00
RexOrCine|away
changed nick to: RexOrCine
15:59
Nira|away
changed nick to: Nira
16:03
danieel
left the channel
16:09
danieel
joined the channel
16:14
se6astian
changed nick to: se6astian|away
16:14
se6astian|away
changed nick to: se6astian
16:25
Y_G
joined the channel
16:45
Y_G
Hi BAndiT1983
16:48
Y_G
I have a response value which is exceeding 1024 ,so should I increase the response buffer or do have any other solution
17:22
se6astian
Hi Y_G, do you have a minute?
17:29
Y_G
hi se6astian ,yes tell me
17:46
se6astian
did BAndiT1983 already chat with you about the wiki documentation of the control daemon CLI examples?
17:46
se6astian
https://wiki.apertus.org/index.php/Control_Daemon#CLI
17:52
Y_G
Yes ,I have went through these ,He asked me to extend these with the functionalities added during the task
17:53
Y_G
I will do once BAndiT1983 approves of the code
17:58
se6astian
What I wanted to propose is adding more examples of commands that are already possible now and do not rely on extending the code
17:58
se6astian
but at the same time making sure they work as expected by testing on the remote beta
17:58
se6astian
in particular chaning image sensor registers
17:58
se6astian
*changing
17:59
se6astian
is that something you could dig into?
18:05
Bertl
off for now ... bbl
18:05
Bertl
changed nick to: Bertl_oO
18:08
Y_G
Checked the code ,there are six functions that are allowed with the un-extended code : `get` and `set` analog_gain,digital_gain ,config_register
18:21
se6astian
yes
18:21
se6astian
the image sensor has 128 register blocks
18:22
se6astian
so examples how to read and set some example registers would be very handy
18:22
se6astian
did you take a look at the image sensor datasheet already?
18:27
Y_G
Yes ,I have gone through it
18:29
se6astian
great
18:30
se6astian
then picking out a few key image sensor parameters and providing examples how to get set them with the control daemon CLI would be great
18:30
se6astian
with full examples of the commands but also of the way the return values are provided
18:30
se6astian
and some explanation what the return values mean exactly and how to interpret them
18:31
Y_G
So do you want specific functions for it for ex "DaemonCLI image_sensor set adc_range 10",or just document what register to set for adc_range
18:31
se6astian
is that something you could work on?
18:31
Y_G
yes definitely,it's a part of my proposal .Will work on it
18:32
se6astian
great
18:32
se6astian
"DaemonCLI image_sensor set adc_range 10" will not work
18:33
se6astian
as you need to provide register index and value
18:33
se6astian
so "just document what register to set for adc_range" is mandatory
18:35
Y_G
if we have a function for adc_range it will automatically know which register to set.
18:36
Y_G
for ex. "DaemonCLI image_sensor set analog_gain 2" ,This automatically sets the register for analog_gain
18:38
Nira
changed nick to: Nira|away
18:47
se6astian
you are right
18:48
se6astian
my focus was on config_register
18:48
se6astian
so "just document what register to set for adc_range" is what I meant
18:48
se6astian
there are two ways how to do the same thing
18:49
se6astian
either its an abstraction already in place
18:49
se6astian
like changing analog gain
18:49
se6astian
or you can look up which image sensor register corresponds to the analog gain
18:49
se6astian
and set the image sensor register directly
18:49
se6astian
thats what I want to have documented on the wiki
18:49
se6astian
and how to do it
18:55
aombk
left the channel
18:55
Y_G
Ok will have a look on what registers look important and will update them accordingly.
18:56
Y_G
Will require for some time for this ,as I have to meet timeline for the first evaluation.
19:35
se6astian
sure
19:35
se6astian
any questions let me know
19:46
RexOrCine
Test
19:48
se6astian
leaving the axiom office now
19:49
se6astian
changed nick to: se6astian|away
20:00
RexOrCine
changed nick to: RexOrCine|away
20:04
BAndiT1983
hi Y_G
20:05
BAndiT1983
what is the problem with 1024 bytes? how is it exceeding?
20:05
BAndiT1983
second thing about index and value for registers, everything is there
20:05
BAndiT1983
the data packet format was extended some while ago and can transfer 2 parameters, setters should use 1 or both, depending on the case
20:07
BAndiT1983
also setters and getters for registers are there also for quite a while, but i had no time to check it, as we had connection problems to HDMI, afterwards my job took some time and will still take a lot of time in the next weeks
20:10
aombk
joined the channel
20:23
Y_G
There is a functionality that requires to capture information about all pac1720 ,The output is as follows : "https://pastebin.com/FDjZNZw8"
20:24
Y_G
The response value is greater than 1024 bytes
20:24
BAndiT1983
pastebin is not longer there
20:24
BAndiT1983
if it exceeds this size, then the approach cannot be correct
20:26
Y_G
https://pastebin.com/MNSUb2EL
20:26
BAndiT1983
ehm, are you really trying to send this data inside flatbuffers packet?
20:28
BAndiT1983
this is exactly the opposite what the daemon communication should do
20:28
BAndiT1983
it would be the task of daemon CLI to format data for output on screen, but the daemon communication should be as small as possible
20:31
Y_G
I have individual functions for getting each row too . But to get the overview of all the sensors would help the user
20:31
Y_G
Even if formating is left to daemon CLI ,it's still a lot of data here
20:32
BAndiT1983
i don't see a necessity to do the formatting on the daemon side
20:32
BAndiT1983
goal is to keep daemon from crashing
20:33
BAndiT1983
have you wrote down the processes and goals, so i can analyse the steps you want to take?
20:35
Y_G
Had a discussion with Bertl this Monday and Tuesday about the goals of breaking down pac1720info.
20:36
BAndiT1983
and what was decided?
20:38
Y_G
My takeaway from that was it would help to have functions for voltages of individual sensors , and the power consumption per rail
20:38
BAndiT1983
ok, but then i don'T understand why you want to create info output in the daemon
20:39
BAndiT1983
there could be a periodical job to grab values, when some script or CLI asks for them, then the cached value is delivered
20:40
BAndiT1983
so oyu would have like a method for 5V_NORTH, 5V_NORTH_AMP or so
20:40
BAndiT1983
*you
20:40
Y_G
It was majorly a test to check I2CAdapter ,to see if it was working properly.Which I suppose does work properly now
20:41
BAndiT1983
if you think it would help to send multiple things, then you have to evaluate how to send proper array and how to use it without hassle while developing modules and while using CLI
20:43
BAndiT1983
another thing, please avoid placing long methods in if/else, you can grab the variables which are different and set values, afterwards call the method using it
20:45
BAndiT1983
smbus.h should be moved to 3rdParty folder, as it seems it was copied from somewhere else
20:45
BAndiT1983
also smbus.cpp of course
20:45
BAndiT1983
what about newer kernels, do we really require this SMBus methods?
20:46
BAndiT1983
word of caution, looks like you've renamed smbus.c to smbus.cpp, first thing is naming, according to C++ it should be SMBus.cpp
20:47
BAndiT1983
second thing is probable problems, because you have to use special keywords sometime to mix C++ and C
20:47
BAndiT1983
-probable +possible
20:49
Dev_
joined the channel
20:51
Dev_
Hello BAndiT1983 ,
20:51
Y_G
I can try without SMBus methods ,should not be a problem.
20:51
BAndiT1983
hi Dev_
20:52
BAndiT1983
Y_G, it'S about evaluating what is the state nowadays, as google was not clear at some points
20:53
Y_G
Have tried going through articles ,all of them suggested to use smbus commands instead of simple read/write
20:53
Y_G
For newer kernels could only find one workaround ,that said just include the functions that are missing from smbus.c
20:54
BAndiT1983
was the reason named, why they were deprecated?
20:57
Y_G
Couldn't find anything on that
20:57
BAndiT1983
https://www.kernel.org/doc/html/latest/driver-api/i2c.html
20:57
BAndiT1983
it only states that I2C supports more things than SMBus, but about the methods it'S not clear
20:59
Dev_
BAndiT1983: I have changed the sharing settings of document . Can u able to check it now
21:00
BAndiT1983
Dev_, yes
21:00
BAndiT1983
will check your allocator tomorrow or while weekend, but have seen quite some problems
21:01
BAndiT1983
have commented on trello and also in the other GDoc file
21:01
Dev_
yes , I saw your comment about IAllocator on trello ,
21:01
Dev_
i will change that
21:02
illwieckz
joined the channel
21:02
BAndiT1983
have you understood why i have commented it?
21:06
BAndiT1983
so, have to leave for today, work awaits tomorrow
21:06
BAndiT1983
if any questions are there, then write here or on trello, will try to do proper code review tomorrow
21:06
BAndiT1983
changed nick to: BAndiT1983|away
21:13
Dev_
The IAllocator is abstract class which contains methods that can be extended for different usecases,
21:16
Dev_
we would be having different implementation for Allocate(), Deallocate() methods for different cases and we can extend the IAllocators methods easily
21:17
Dev_
i will add IAllocator again
21:17
Dev_
working on your comments on Gdoc document now ..
21:48
Dev_
left the channel
22:00
comradekingu
joined the channel
22:12
illwieckz_
joined the channel
22:15
illwieckz
left the channel
22:46
Y_G
left the channel