Thorben T.

``` $ cat etc/vzlogger.conf.direction_from_status { "verbosity": 15, "retry": 0, "meters": [{ "enabled": true, "allowskip": false, "protocol": "sml", "device": "/tmp/EMH-eHZ-HW8E2A5L0EQ2P_2kw_imp.bin", //"device": "/tmp/EMH-eHZ-HW8E2A5L0EQ2P_8kw_exp.bin", "direction_from_status": true, }] } [~/vzlogger]$ ./src/vzlogger -c etc/vzlogger.conf.direction_from_stat us...

@ednax: > does that mean, that vzlogger will support this in a future version? you can simply use the code from my branch, i posted the command to download it...

> > is it ok if i add your dumps to https://github.com/devZer0/libsml-testing ? > > Does these files contain any security relevant data? > If they don't contain sensitive data...

@SirkoVZ: enjoy, good that it's of use. as said, the code needs some improvements before i would consider it ready for merge to master. see https://github.com/volkszaehler/vzlogger/pull/476#pullrequestreview-627218150

@SirkoVZ: > I would appreciate if it would be possibe to log the negative and positive values into two different channels. i think that's the thing @J-A-U already hinted at?...

looks good, but not tested yet, wonder if "field_name" is a good idea for the config parameter, as it's influxdb-specific. (or maybe it can be useful for other APIs too?...

yes, it's about the name, if it's ONLY used by influxdb, i'd think "influxdb_field_name" or such would be more appropriate. #420 mentions uuid, but i think it's really about the...

> "influxdb_field_name" is a long string but if we introduced it with the short name, we would have to add comments everywhere, explaining that it's only for influxdb, to avoid...

``` for (it = buf->begin(); it != buf->end(); ++it) { Reading &r = *it; if (!r.deleted()) { ``` https://github.com/volkszaehler/vzlogger/blob/master/src/local.cpp#L87 (the method then crashes dereferencing `this->_deleted`.) i can't tell if it's...

i don't think reloading the configuration on sighup is sensible to implement. we either need to tear down and rebuild all state (might aswell restart), or we would need a...