Feature request - custom annotation

mikmonken · February 26, 2015, 9:30am

realise that this was raised a few months ago on the old forum - http://forum.brewpi.com/discussion/149/add-custom-annotations-to-graph , but just wondered whether there had been any progress on adding in custom annotation functionality for tracking gravity at brewing.

Not that I’ve rigged my fridge light to function through brewpi but if on opening door it triggered the opportunity to enter a gravity reading that would be quite good.

i.e. you open the fridge door (i’m guessing the only reason people open the fridge when using brewpi is to check gravity) it creates a time stamp including beer temp and option to manually add gravity reading, which could then be automatically adjusted to give temperature adjusted reading?

for me personally i guess as my set up isn’t rigged to trigger the door light i’d like to be able to enter it all manually.

is any of that feasible?

Elco · February 27, 2015, 11:30pm

I think the main reason is curiousity! I only check gravity near the end.

The easiest way to hack this in is to log a data point in the script, with the annotation and the last known temperatures:

The line is added to the json file here:

github.com

BrewPi/brewpi-script/blob/master/brewpi.py#L707


        response_str = json.dumps(response)
        conn.send(response_str)
    elif messageType == "resetController":
        logMessage("Resetting controller to factory defaults")
        bg_ser.writeln("E")
    else:
        logMessage("Error: Received invalid message on socket: " + message)


    if (time.time() - prevTimeOut) < serialCheckInterval:
        continue
    else:
        # raise exception to check serial for data immediately
        raise socket.timeout


except socket.timeout:
    # Do serial communication and update settings every SerialCheckInterval
    prevTimeOut = time.time()


    if hwVersion is None:
        continue  # do nothing with the serial port when the controller has not been recognized

Triggered by a message from the web interface.

We will switch to data logging into influxDB in the near future, that is why we are not investing time in implementing it in the current code.

mikmonken · February 28, 2015, 9:15am

Being pretty much illiterate in the art of programming, I may hold out for influxDB rather than risk messing up what currently works pretty well, I’ll stick to writing gravity readings on my fridge.