n0bel / PiClock

A Fancy Clock built around a monitor and a Raspberry Pi
MIT License
566 stars 182 forks source link

The top line does not display the days of the week with regional settings. #206

Closed xenon462 closed 1 year ago

xenon462 commented 3 years ago

1

In the PyQtPiClock.py file I added the line

ds = ds.decode('utf-8') ds2 = ds2.decode('utf-8')

date decode

The top line worked correctly. 2

3

How do I fix the days of the week in the column on the right?

My settings: set

BertLindeman commented 3 years ago

Disclaimer: Not in topcondition at the moment, no guarantee what so ever . . . .

Looks like you use ClimaCell The 3 daily forcasts are formatted in wxfinished_cc2 Near line 909:

        day = fl.findChild(QtGui.QLabel, "day")
        day.setText("{0:%A %I:%M%p}".format(
            dateutil.parser.parse(f['observation_time']['value'])
            .astimezone(tzlocal.get_localzone())))

You could try to fiddle a decode in there.

Similar near line 969.

xenon462 commented 3 years ago

Added "decode ('utf-8')" next to line 909 decode2

The right column only displays the top three rows correctly. file5

The other six days of the week are not displayed.

BertLindeman commented 3 years ago

Well done, really.

You missed my last line:

Similar near line 969.

That is in 'wxfinished_cc3'

xenon462 commented 3 years ago

Made changes on line 976 section 'wxfinished_cc3'

decode2day

All days of the week work file6

How do I ask Kevin to make these fixes to the main code of the program?

BertLindeman commented 3 years ago

The procedure would be to edit Clock/PyQtPiClock.py There is a pencil icon near the top on the right side

Edit your changes and you should be able to create a pull request.

And then it is up to Kevin to accept or reject.

It's no rocket science but github can be confusing at times.

xenon462 commented 3 years ago

I can't see a pencil. I will search now. Thanks for the help.

ScottChapman commented 3 years ago

You sure you don't see this: image Maybe you aren't logged in?

xenon462 commented 3 years ago

found a pencil. I'll try to fix it now.

BertLindeman commented 3 years ago

@xenon462 has added Pull Request #207 for this issue