| 01:27 | Bertl_oO | off to bed now ... have a good one everyone!
|
| 01:27 | Bertl_oO | changed nick to: Bertl_zZ
|
| 02:05 | calle__ | joined the channel |
| 02:08 | mumptai_ | left the channel |
| 09:11 | BAndiT1983|away | changed nick to: BAndiT1983
|
| 09:24 | david[m]17 | joined the channel |
| 10:47 | calle__ | left the channel |
| 10:48 | mumptai | joined the channel |
| 10:56 | comradekingu | left the channel |
| 11:02 | comradekingu | joined the channel |
| 11:45 | ronald[m] | joined the channel |
| 12:29 | Bertl_zZ | changed nick to: Bertl
|
| 12:29 | Bertl | morning folks!
|
| 12:29 | BAndiT1983 | hi
|
| 12:30 | se6ast1an | good day
|
| 14:56 | panintended | Hi all
|
| 14:56 | panintended | BAndiT1983:
|
| 14:57 | panintended | Regarding the Observer pattern, will the attributes that Observers subscribe to be static, or would you expect it to change during runtime?
|
| 14:58 | BAndiT1983 | hi panintended, how are you?
|
| 14:59 | BAndiT1983 | they will be static, as remote will only handle the ones which are available in the firmware
|
| 14:59 | panintended | To clarify: would you expect the pool of attributes to change during runtime (attributes added/removed from the pool). their values are expected to change of course :)
|
| 15:00 | BAndiT1983 | later we can do something like request all available ones from the camera when remote starts
|
| 15:00 | BAndiT1983 | nah, not really, but maybe Bertl or se6ast1an have different ideas about that
|
| 15:01 | panintended | I'm doing well, I'm afraid it's going just weekends for me for AXIOM from here on (better than nothing though). Yourself?
|
| 15:02 | panintended | I see, thanks for the clarification!
|
| 15:02 | panintended | I'm trying to cut down as much as I can on memory allocation/copying, and this just came to me :)#
|
| 15:02 | BAndiT1983 | trying to push things forward, but have to do some time management as private life also needs attention
|
| 15:03 | panintended | of course :)
|
| 15:03 | BAndiT1983 | as long as you don't use new() ;)
|
| 15:04 | panintended | really? no new()?! ;)
|
| 15:05 | panintended | actually it was about copying over the C-strings in which attribute names would be stored in when an observer subscribes, etc. But it looks like that won't be necessary
|
| 15:06 | BAndiT1983 | would enum class help? but i think that we need to register things a bit dynamically, so maybe string would be ok
|
| 15:07 | BAndiT1983 | new() is possible, but it consumes additional memory and heap has to be adjusted in the linker, we know our requirements beforehand, that's why dynamic memory was ditched
|
| 15:08 | panintended | yeah, and in general dynamic memory is not great for real-time systems
|
| 15:09 | panintended | Well an enum would really simplify things, yes. If we're going with a static attribute pool, we might as well ditch strings completely
|
| 15:11 | panintended | Do you mean something like enum Attribute { WHITE_BALANCE, EXPOSURE, <OTHER CAMERA CONFIG>, ... };
|
| 15:11 | BAndiT1983 | yep
|
| 15:12 | BAndiT1983 | but c++11 enum please, aka enum class
|
| 15:12 | BAndiT1983 | this would avoid collisions, naming schema is a bit different there, but you can also use the classic enum naming
|
| 15:14 | panintended | ok, cool
|
| 15:14 | panintended | thanks
|
| 15:16 | BAndiT1983 | no problem
|
| 16:00 | BAndiT1983 | changed nick to: BAndiT1983|away
|
| 18:02 | Bertl | off for now ... bbl
|
| 18:02 | Bertl | changed nick to: Bertl_oO
|
| 18:02 | panintended | Hi BAndiT1983. I've pushed to my fork the framework for the observer pattern. Can you have a look? Be sure to pull from branch "feature_T1203"
|
| 18:03 | panintended | I'll be out for an hour or two, so I'll check any comments after that. Thanks!
|
| 18:06 | panintended | https://github.com/panintended/AXIOM-Remote/tree/feature_T1203
|
| 18:25 | se6ast1an | <BAndiT1983> nah, not really, but maybe Bertl or se6ast1an have different ideas about that <- no different ideas from me, since the parameters are dictated by the hardware everything is really "set in stone" :)
|
| 18:25 | se6ast1an | like the image sensor parameters
|
| 18:26 | BAndiT1983|away | changed nick to: BAndiT1983
|
| 18:27 | BAndiT1983 | panintended: will try to free some time and check it at some point
|
| 18:27 | se6ast1an | one thing we could discuss is how we could handle new hardware with the existing Remote in the future, like the upcoming AXIOM Micro R3 hardware - but for me a simple firmware upgrade for the remote is the answer which extends the static list where required
|
| 18:27 | BAndiT1983 | certain remote firmware version for certain hardware/firmware version of beta
|
| 18:28 | se6ast1an | yeah or just the latest firmware is able to support the latest cameras
|
| 18:28 | se6ast1an | and understands which camera it is connected to
|
| 18:30 | BAndiT1983 | should also work, but we should keep probable differences in mind and handle them accordingly, e.g. missing parameters on certain camera models
|
| 18:30 | BAndiT1983 | we should just try to follow a route which is flexible enough and adjust the handling later
|
| 18:33 | se6ast1an | agreed
|
| 18:34 | BAndiT1983 | but the point with missing params is easily doable, if we can get the list of available params of the camera and disable unavailable entries, they can still be there but greyed out, or just not shown at all
|
| 18:39 | se6ast1an | yes, my idea would be to show the name of the recognized connected camera somewhere and then only display the parameters that this model supports
|
| 18:49 | BAndiT1983 | already having a plan for some time how to test it without actual hardware, but the implementation has probably done by someone else, as i'm still not progressing that well with my tasks at the moment, as other things are coming in-between again and again
|
| 19:28 | se6ast1an | fingers crossed
|
| 20:23 | bluez_[m] | Hey Bertl! Please ping me once you are back... I have an issue I need to discuss...
|
| 21:26 | BAndiT1983 | changed nick to: BAndiT1983|away
|
| 22:04 | BAndiT1983|away | changed nick to: BAndiT1983
|
| 22:06 | BAndiT1983 | changed nick to: BAndiT1983|away
|
| 23:05 | LordVan | left the channel |
| 23:17 | LordVan | joined the channel |
| 23:59 | alexML | left the channel |
| 00:06 | bluez_[m] | <bluez_[m] "Hey Bertl! Please ping me once y"> okay i solved the issue ':D ....will discuss tomorrow... off to bed now...
|
| 00:08 | alexML | joined the channel |