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

Current Hour Icon empty #66

Closed
robertfraefel opened this issue Nov 12, 2021 · 37 comments
Closed

Current Hour Icon empty #66

robertfraefel opened this issue Nov 12, 2021 · 37 comments
Labels
bug Something isn't working

Comments

@robertfraefel
Copy link

I have often the problem (several times a day) that the datapoint ICON_URL_COLOR is empty (dummy) and therefore no picture is displayed in my VIS.

Screenshot 2021-11-12 at 07 27 57

@baerengraben
Copy link
Owner

I saw this once on my system too. But I could not reproduce it. Here it happend just once.
If this happens to you often, I am very interested in your log. Can you set the adapter to "debug", reproduce the problem and then provide the log?

Additionally:
With which hardware do you operate the ioBroker?

@robertfraefel
Copy link
Author

robertfraefel commented Nov 18, 2021 via email

@baerengraben
Copy link
Owner

@robertfraefel : Have you forgot the log? It seems that it is not attached.

@robertfraefel
Copy link
Author

robertfraefel commented Nov 19, 2021 via email

@baerengraben
Copy link
Owner

nope :)

Can you attach it directly in #66 ?

@robertfraefel
Copy link
Author

Attached the log.
iobroker.2021-11-17.log.gz

@baerengraben
Copy link
Owner

Ok. I asume, these variables have to be in the async-function:

var date = new Date();

Fix is comming soon.

baerengraben pushed a commit that referenced this issue Dec 31, 2021
@baerengraben
Copy link
Owner

@robertfraefel : I did a fix for this issue. since I can't reproduce the problem directly on my computer, I would be glad if you could test it. For that you would have to install the swissweather adapter directly from Github and test it. Could you do that?

@robertfraefel
Copy link
Author

Thanks, just downloaded from github and will check and let you know.

@robertfraefel
Copy link
Author

Still have the issue form time to time.
Screenshot 2022-01-03 at 15 58 04

@baerengraben
Copy link
Owner

@robertfraefel : Thanks for testing.

This is really interesting. Just to be sure, when you reinstalled the adapter from Github, did you completely remove the adapter before?

  • Remove instance from ioBroker
  • Delete adapter from ioBroker
  • Install new adapter from Github
  • Install adapter instance

baerengraben pushed a commit that referenced this issue Jan 4, 2022
@robertfraefel
Copy link
Author

Yes, I completely removed the adapter and install it new with the github link. It shows still the same Version.
Can I check somehow if the fix was pulled?

@baerengraben
Copy link
Owner

baerengraben commented Jan 4, 2022

@robertfraefel I made an adjustment in the code today for this purpose. Now "dummyicon" would be the output instead of "dummy". Can you please reinstall the adapter and check whether "dummyicon" is now the output?

@robertfraefel
Copy link
Author

Screenshot 2022-01-05 at 10 26 18

@baerengraben
Copy link
Owner

OK. Here is the good news. I finally can reproduce it:
Bildschirmfoto vom 2022-01-06 23-01-20

The bad news is: still asking myself "WHY?!" ;)

but I'll figure it out...

baerengraben pushed a commit that referenced this issue Jan 6, 2022
@baerengraben baerengraben added the bug Something isn't working label Jan 6, 2022
baerengraben pushed a commit that referenced this issue Jan 7, 2022
@robertfraefel
Copy link
Author

If I can help just let me know...

baerengraben pushed a commit that referenced this issue Jan 7, 2022
baerengraben pushed a commit that referenced this issue Jan 8, 2022
baerengraben pushed a commit that referenced this issue Jan 8, 2022
baerengraben pushed a commit that referenced this issue Jan 8, 2022
baerengraben pushed a commit that referenced this issue Jan 8, 2022
@baerengraben
Copy link
Owner

@robertfraefel : I have "redesigned" the code. Can you please install the adapter from github again and test it?

@robertfraefel
Copy link
Author

currently testing since this morning...

@robertfraefel
Copy link
Author

Screenshot 2022-01-10 at 18 21 34
Sorry.....

baerengraben pushed a commit that referenced this issue Jan 10, 2022
@baerengraben
Copy link
Owner

@robertfraefel : Unfortunately, I can rarely reproduce this error. So, I have to count on your testing help more often than usual. Hope this is ok for you.
Can you please reinstall adapter from github and test the adapter with debug mode? If the error occurs again, please also attach the log into this issue.

@robertfraefel
Copy link
Author

Done, testing started, no errors
Screenshot 2022-01-11 at 07 38 08

@robertfraefel
Copy link
Author

until now no further dummy icons!

Screenshot 2022-01-11 at 20 53 27

@baerengraben
Copy link
Owner

wooohooo :)
I hope it stays like this cross fingers

@robertfraefel
Copy link
Author

until now only once....
Screenshot 2022-01-12 at 17 50 27

@baerengraben
Copy link
Owner

ok. can you please provide the debug-log?

@robertfraefel
Copy link
Author

unfortunately did not switch on debug mode. Did now and have to wait for the next dummy icon event. By the way, the only one I have so fare is the one from above.

@baerengraben
Copy link
Owner

Ok, no problem and thanks for your help! ..it looks like we are on the "right track" now

@robertfraefel
Copy link
Author

Sorry for my late response. Was few days skiing and had difficulties to download the logfile from the iobroker server. The last dummy event I see on 16/01/2022 22:00:00.222

I tried to attach the log file here but I guess it is to big.

What I see for this time in the log is:
2022-01-16 22:00:00.213 - �[32minfo�[39m: swiss-weather-api.0 (1186) update current hour...
2022-01-16 22:00:00.213 - �[34mdebug�[39m: swiss-weather-api.0 (1186) Defining local current_hour variables...
2022-01-16 22:00:00.213 - �[34mdebug�[39m: swiss-weather-api.0 (1186) Updating local current_hour variables...
2022-01-16 22:00:00.215 - �[34mdebug�[39m: swiss-weather-api.0 (1186) ...and now add updated variables to current_hour objects.

@robertfraefel
Copy link
Author

do you have any other way to send you big files?

@baerengraben
Copy link
Owner

Thanks for your feedback. These are the only log entries at this time from the swiss-weather-adapter?

@robertfraefel
Copy link
Author

iobroker.2022-01-16_ext.log
I made an extract of the log. Please have a look.

baerengraben pushed a commit that referenced this issue Jan 21, 2022
@baerengraben
Copy link
Owner

@robertfraefel : I did another fix. Can you please reinstall adapter from github and test the adapter again with debug mode?

@robertfraefel
Copy link
Author

Until now no new dummy icons discovered!

@baerengraben
Copy link
Owner

Great :) So let's wait until next friday and see if any more dummy icons show up. If no other dummy icons appear, I will create a new release. Can you please check it next week and give me feedback?

Thank you very much for your help! :)

@robertfraefel
Copy link
Author

no other dummy icon showed up!

@baerengraben
Copy link
Owner

Great, new Release is landing... ;)

baerengraben pushed a commit that referenced this issue Feb 7, 2022
@baerengraben
Copy link
Owner

Fixed with Version 1.0.3
Thanks for reporting and testing!

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

2 participants