pynmea2 icon indicating copy to clipboard operation
pynmea2 copied to clipboard

nmea_utils: Added timezone

Open barbacbd opened this issue 3 years ago • 10 comments

** UTC timezone is applied when returning the information from datetime.

Fix applied for #100

barbacbd avatar Jul 11 '22 14:07 barbacbd

Coverage Status

Coverage increased (+0.002%) to 98.346% when pulling 1cc09faf9cd584b0b3b35ce5d2b134bfbaaf8a64 on barbacbd:fix_utc_timezone into a0d60b62ab2242af7b3a64c415947eb52a3638a5 on Knio:master.

coveralls avatar Jul 11 '22 14:07 coveralls

Coverage Status

Coverage increased (+0.002%) to 98.346% when pulling 1cc09faf9cd584b0b3b35ce5d2b134bfbaaf8a64 on barbacbd:fix_utc_timezone into a0d60b62ab2242af7b3a64c415947eb52a3638a5 on Knio:master.

coveralls avatar Jul 11 '22 14:07 coveralls

Coverage Status

Coverage increased (+0.02%) to 98.38% when pulling 95035c30b50af7d91486ca6299fa11e3f1fa7e93 on barbacbd:fix_utc_timezone into 988c297ce82d976db9094b435a1aa290e7d5b9ed on Knio:master.

coveralls avatar Jul 11 '22 14:07 coveralls

@Knio Looks like another python incompatibility. I fixed that, so can you give this a once over (or twice) for me?

barbacbd avatar Jul 27 '22 18:07 barbacbd

is there a way to only require the install dep on py2.7, and use datetime.timezone.utc on py3? I don't want to introduce unnecessary 3rd party dependencies

Knio avatar Jul 28 '22 22:07 Knio

Sure I can do some import "magic" If you want. Some people want to use the same deps for everything and some want what you want. You never know until you try right! I will make that change .

barbacbd avatar Jul 29 '22 17:07 barbacbd

Actually it looks like it was python 2.7, and < 3.6

barbacbd avatar Jul 29 '22 17:07 barbacbd

@Knio hopefully I have what you wanted pushed here. May need to rerun the tests as I pushed something prematurely earlier. The tests pass in 2.7, 3.6, and 3.8 on my system .

barbacbd avatar Jul 29 '22 19:07 barbacbd

@Knio is this one good to go or do you need more changes ?

barbacbd avatar Aug 04 '22 17:08 barbacbd

@Knio can we close this one or merge or other haha? I have it still sitting in my queue

barbacbd avatar Oct 07 '22 15:10 barbacbd

@Knio this has conflicts with the main branch. Looks like you may have added something similar. Is this safe to close ?

barbacbd avatar Nov 10 '22 19:11 barbacbd

Hey, yes I did. Are you able to confirm if what's in master solves the problem for you?

Knio avatar Nov 17 '22 07:11 Knio

Should work for me.

barbacbd avatar Nov 17 '22 20:11 barbacbd