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

Error on Parse SolarLog JSON #2

Open
riodda opened this issue Mar 21, 2023 · 4 comments
Open

Error on Parse SolarLog JSON #2

riodda opened this issue Mar 21, 2023 · 4 comments

Comments

@riodda
Copy link

riodda commented Mar 21, 2023

Hi, i manage to get to the solar log json interface, i deploy the flow but i'm having an error but data is sent to mqtt correctly.
image

@riodda
Copy link
Author

riodda commented Mar 21, 2023

The other thing that I've noticed is that when i look into the db the value is the timestamp instead of the actual valute
image
image

@Sineos
Copy link
Owner

Sineos commented Mar 21, 2023

Hi, i manage to get to the solar log json interface, i deploy the flow but i'm having an error but data is sent to mqtt correctly.

The function was very sloppy. Not a real error but NR rightfully complained. Fixed in latest commit.

The other thing that I've noticed is that when i look into the db the value is the timestamp instead of the actual valute

If you work with InfluxDB you will want to use the UTC timestamp or you easily create a mess in your DB.

@riodda
Copy link
Author

riodda commented Mar 27, 2023

The other thing that I've noticed is that when i look into the db the value is the timestamp instead of the actual valute

If you work with InfluxDB you will want to use the UTC timestamp or you easily create a mess in your DB.

Of course you want to have the timestamp, but not only the timestamp as it's happening now, I have no value and only time stamp, i for sure did some mistake but can't figure out what mistake

@Sineos
Copy link
Owner

Sineos commented Mar 31, 2023

image

Just check with the debug nodes. From your screenshot above the values seems OK.
IIRC I have chosen the format to be directly importable into InfluxDB via node-red-contrib-influxdb but in any case it is easy to change in the JS code if you want to have a different format.

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

No branches or pull requests

2 participants