Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add last_updated and last_changed fields for debugging purposes #1

Open
dbrgn opened this issue Mar 26, 2015 · 14 comments
Open

Add last_updated and last_changed fields for debugging purposes #1

dbrgn opened this issue Mar 26, 2015 · 14 comments

Comments

@dbrgn
Copy link
Member

dbrgn commented Mar 26, 2015

last_updated wird bei jedem erfolgreichen POST vom sensor updated.

last_changed wird immer geändert wenn sich die anzahl personen ändert.

@rnestler
Copy link
Member

last_updated könnte man direkt über die mtime des files herausfinden: datetime.datetime.fromtimestamp(os.path.getmtime('people.txt')) bzw. für UTC datetime.datetime.utcfromtimestamp(os.path.getmtime('people.txt'))

@rnestler
Copy link
Member

So ala b27dd9b

@dbrgn
Copy link
Member Author

dbrgn commented Mar 29, 2015

Guter Punkt :) Werde ich gleich mergen.

Die Frage ist noch, wie die Daten in der JSON Response codiert werden.

@dbrgn
Copy link
Member Author

dbrgn commented Mar 29, 2015

Last change:

{
    "state": {
        "lastchange": 123123123
    }
}

Last update:

{
    "events": [
        {
            "name": "Presence Switches",
            "type": "updated_status",
            "timestamp": 123123123
        }
    ]
}

@rnestler
Copy link
Member

Gemäss der Doku:

lastchange (number)
Description
The Unix timestamp when the space status changed most recently

Unser timestamp zeigt aber die Änderung der Anzahl Personen, nicht des Status.

@dbrgn
Copy link
Member Author

dbrgn commented Mar 29, 2015

Das ist doch auch eine Statusänderung :)

@rnestler
Copy link
Member

Ja aber keine Änderung des ''state'' Objekts, aber ich bin wahrscheinlich ein wenig zu pedantisch 😉

@dbrgn
Copy link
Member Author

dbrgn commented Mar 29, 2015

In unserem Fall schon, da sich die message im "state" Objekt ändert. Hast du eine bessere Idee?

@rnestler
Copy link
Member

Ich hätte es ebenfalls in "events" eingetragen, habe aber mittlerweile meine Meinung geändert 😄 Wir ändern ja tatsächlich unseren Status.

@rnestler
Copy link
Member

Für last_changed müssen wir uns wohl den alten Wert merken. Gibt es dafür eleganteres als eine globale Variable?

@dbrgn
Copy link
Member Author

dbrgn commented Mar 29, 2015

Am besten diskutieren wir das morgen :) Entweder ein zweites File, oder gleich eine file based database. Oder eben globale variablen (yuck).

@rnestler
Copy link
Member

Hier mal mit globals 237b5b9 als Diskussionsgrundlage für Morgen ;)

@dbrgn dbrgn added the rust label Apr 11, 2015
@rnestler
Copy link
Member

This ticket did concern the python implementation, didn't it? So why the rust label?

@dbrgn
Copy link
Member Author

dbrgn commented Nov 13, 2015

Because we still want that feature :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants