PiClock icon indicating copy to clipboard operation
PiClock copied to clipboard

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

Open xenon462 opened this issue 4 years ago • 9 comments

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

xenon462 avatar Jan 05 '21 12:01 xenon462

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.

BertLindeman avatar Jan 05 '21 16:01 BertLindeman

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.

xenon462 avatar Jan 05 '21 16:01 xenon462

Well done, really.

You missed my last line:

Similar near line 969.

That is in 'wxfinished_cc3'

BertLindeman avatar Jan 05 '21 16:01 BertLindeman

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?

xenon462 avatar Jan 05 '21 17:01 xenon462

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.

BertLindeman avatar Jan 05 '21 17:01 BertLindeman

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

xenon462 avatar Jan 05 '21 17:01 xenon462

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

ScottChapman avatar Jan 05 '21 18:01 ScottChapman

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

xenon462 avatar Jan 05 '21 18:01 xenon462

@xenon462 has added Pull Request #207 for this issue

BertLindeman avatar Jan 05 '21 18:01 BertLindeman