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

Priceinfo issue, wrong response from entso-e? #35

Closed
joe-ave opened this issue Aug 5, 2023 · 9 comments
Closed

Priceinfo issue, wrong response from entso-e? #35

joe-ave opened this issue Aug 5, 2023 · 9 comments
Assignees
Labels
bug Something isn't working

Comments

@joe-ave
Copy link

joe-ave commented Aug 5, 2023

Hello Olli,

The priceinfo from entso-e seems wrong, priceinfo is there when I look at the serial monitor but timeseries is invalid.
I created a new issue instead of continuing at the old one since this is probably a new behaviour.

#29 (comment)

It´s the same problem at both my newer 0.99.0-beta4.3344 - 2023-08-04 14:54:34 and my old 0.92.0-rc2.
My old 0.92 says the date is 1905 in the pricegraf.

image

//Jonas

@joe-ave
Copy link
Author

joe-ave commented Aug 6, 2023

Price data for countries provided from Elering ( Estonia, Finland, Lithuania and Latvia ) works but not for rest of Europe provided from Entso-e.

//Jonas

@Olli69
Copy link
Contributor

Olli69 commented Aug 6, 2023

Hi Jonas,
Thanks for the report! Arska is pretty sensitive to xml document changes in price query (or probably some kind of http communication changes in this case) . That is because it cannot handle big xml document as a whole, but reads it in pieces.

Anyway, I made a quick fix version (0.99.0-alfa10) which is now published and after initial testing seems to fix this problem. If you need to apply it to an older version, you can add 5 lines (starting with a comment // line is complete if ends with >,6.8.2023 /OR ) to a right place in function read_http11_line

Olli

@Olli69 Olli69 self-assigned this Aug 6, 2023
@Olli69 Olli69 added the bug Something isn't working label Aug 6, 2023
@Apulanta
Copy link

Apulanta commented Aug 6, 2023

Hi Olli,
I've been using my modbus server modified version successfully for almost a year, for this last update you made, I couldn't find instructions on how and where to add those five lines of program code to that Arska source code?

Regards, Rane

@joe-ave
Copy link
Author

joe-ave commented Aug 6, 2023

Hello Olli,

Thanks for the fix!, I did however not get my old .92 version to work with the mod of the five lines, so I reflashed with the latest fw for that one allso.
I only lost the onboard led status blink and the 30 minute offsetfix to influxdb but I’ll fix that later.

//Jonas

@Olli69
Copy link
Contributor

Olli69 commented Aug 7, 2023

Hello Rane @Apulanta,
Starting from line 3668 main.cpp , so changes are in the latest devel-ui branch. But there have been quite a lot changes in last year, so it is possible that you should take latest code and add there your modbus changes.

Olli

@Olli69
Copy link
Contributor

Olli69 commented Aug 7, 2023

Hi Jonas @joe-ave,
If I remember right current code does not have that 30 min offset any more - so it should be the way you suggested. But there have been very little testing with influx with the current version, but if there are any issues I'll try to fix them. Anyway now you see the closest future and history on the front page, so Influx is not necessarily needed that much on daily basis.

Olli

@Apulanta
Copy link

Apulanta commented Aug 7, 2023

Hi Olli,
Thank you very much for your help and instructions, I looked at your message and that new 0.99 source code has changed enormously and grown to the 0.92 stable version I used so much that I don't know how to fix that old 0.92 code.

I downloaded this 0.99 alpha10 code and I'm going to add to it (if I can :) ) the modbus server addition I modified..

Regards, Rane

@joe-ave
Copy link
Author

joe-ave commented Aug 7, 2023

Hello Olli,

yes the 30 minute offset is gone now so it looks good when using "step after" interpolation, I did not notice at first!

I moved from the influxdb cloud solution to a local installation running on a raspberry pi together with grafana, mostly because the "europe.west1" cloud were to be shut down and I needed to move anyway. No big issues so far.

//Jonas

@Olli69
Copy link
Contributor

Olli69 commented Aug 10, 2023

Fixed in the latest version 0.99 alpha10 (code in branch devel-ui). Now closing - please comment/open the issue if needed.

@Olli69 Olli69 closed this as completed Aug 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants