search

pi-top / pi-top-Python-SDK

pi-top's Python SDK (pitop package)
Apache License 2.0
27 stars 4 forks source link

Simulate miniscreen #578

Closed angusjfw closed 1 year ago

angusjfw commented 1 year ago
Status Ticket/Issue
Ready Ticket
lgtm-com[bot] commented 1 year ago

This pull request introduces 4 alerts when merging de41307f005ebf24bf3ac59ebf2294ba783afe3f into 4d5b7865dac595a1ae55367f62ab64f2ed814dd6 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 4 alerts when merging 947833241b15e4925bb927a2ec0ce759a5ccd3de into 19a2b02a4e0881ae450d641c7351b19cb432ae55 - view on LGTM.com

new alerts:

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine :gear: that powers LGTM.com. For more information, please check out our post on the GitHub blog.

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert when merging 76939a0ebf47864a54244f14f246ef6fc4a17005 into 19a2b02a4e0881ae450d641c7351b19cb432ae55 - view on LGTM.com

new alerts:

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine :gear: that powers LGTM.com. For more information, please check out our post on the GitHub blog.

codecov[bot] commented 1 year ago

Codecov Report

Base: 60.53% // Head: 60.39% // Decreases project coverage by -0.13% :warning:

Coverage data is based on head (d342332) compared to base (19a2b02). Patch coverage: 26.16% of modified lines in pull request are covered.

:exclamation: Current head d342332 differs from pull request most recent head 7c6624e. Consider uploading reports for the commit 7c6624e to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #578 +/- ## ========================================== - Coverage 60.53% 60.39% -0.14% ========================================== Files 147 149 +2 Lines 7323 7401 +78 ========================================== + Hits 4433 4470 +37 - Misses 2890 2931 +41 ``` | Flag | Coverage Δ | | |---|---|---| | unittests | `60.39% <26.16%> (-0.14%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pi-top#carryforward-flags-in-the-pull-request-comment) to find out more. | [Impacted Files](https://codecov.io/gh/pi-top/pi-top-Python-SDK/pull/578?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pi-top) | Coverage Δ | | |---|---|---| | [...ulation/pitop/simulation/virtual\_hardware/pitop.py](https://codecov.io/gh/pi-top/pi-top-Python-SDK/pull/578/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pi-top#diff-cGFja2FnZXMvc2ltdWxhdGlvbi9waXRvcC9zaW11bGF0aW9uL3ZpcnR1YWxfaGFyZHdhcmUvcGl0b3AucHk=) | `12.12% <12.12%> (ø)` | | | [packages/simulation/pitop/simulation/sprites.py](https://codecov.io/gh/pi-top/pi-top-Python-SDK/pull/578/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pi-top#diff-cGFja2FnZXMvc2ltdWxhdGlvbi9waXRvcC9zaW11bGF0aW9uL3Nwcml0ZXMucHk=) | `38.59% <15.38%> (-9.23%)` | :arrow_down: | | [...ulation/pitop/simulation/virtual\_hardware/fonts.py](https://codecov.io/gh/pi-top/pi-top-Python-SDK/pull/578/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pi-top#diff-cGFja2FnZXMvc2ltdWxhdGlvbi9waXRvcC9zaW11bGF0aW9uL3ZpcnR1YWxfaGFyZHdhcmUvZm9udHMucHk=) | `17.64% <17.64%> (ø)` | | | [packages/simulation/pitop/simulation/simsprite.py](https://codecov.io/gh/pi-top/pi-top-Python-SDK/pull/578/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pi-top#diff-cGFja2FnZXMvc2ltdWxhdGlvbi9waXRvcC9zaW11bGF0aW9uL3NpbXNwcml0ZS5weQ==) | `37.66% <35.29%> (-0.80%)` | :arrow_down: | | [...tion/pitop/simulation/virtual\_hardware/\_\_init\_\_.py](https://codecov.io/gh/pi-top/pi-top-Python-SDK/pull/578/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pi-top#diff-cGFja2FnZXMvc2ltdWxhdGlvbi9waXRvcC9zaW11bGF0aW9uL3ZpcnR1YWxfaGFyZHdhcmUvX19pbml0X18ucHk=) | `57.14% <50.00%> (+3.80%)` | :arrow_up: | | [packages/simulation/pitop/simulation/simulation.py](https://codecov.io/gh/pi-top/pi-top-Python-SDK/pull/578/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pi-top#diff-cGFja2FnZXMvc2ltdWxhdGlvbi9waXRvcC9zaW11bGF0aW9uL3NpbXVsYXRpb24ucHk=) | `50.45% <75.00%> (+0.92%)` | :arrow_up: | | [...ges/simulation/pitop/simulation/images/\_\_init\_\_.py](https://codecov.io/gh/pi-top/pi-top-Python-SDK/pull/578/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pi-top#diff-cGFja2FnZXMvc2ltdWxhdGlvbi9waXRvcC9zaW11bGF0aW9uL2ltYWdlcy9fX2luaXRfXy5weQ==) | `100.00% <100.00%> (ø)` | | | [packages/system/pitop/system/pitop.py](https://codecov.io/gh/pi-top/pi-top-Python-SDK/pull/578/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pi-top#diff-cGFja2FnZXMvc3lzdGVtL3BpdG9wL3N5c3RlbS9waXRvcC5weQ==) | `100.00% <100.00%> (ø)` | | | [packages/miniscreen/pitop/miniscreen/miniscreen.py](https://codecov.io/gh/pi-top/pi-top-Python-SDK/pull/578/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pi-top#diff-cGFja2FnZXMvbWluaXNjcmVlbi9waXRvcC9taW5pc2NyZWVuL21pbmlzY3JlZW4ucHk=) | `73.33% <0.00%> (+25.00%)` | :arrow_up: | | [...ages/core/pitop/core/mixins/supports\_miniscreen.py](https://codecov.io/gh/pi-top/pi-top-Python-SDK/pull/578/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pi-top#diff-cGFja2FnZXMvY29yZS9waXRvcC9jb3JlL21peGlucy9zdXBwb3J0c19taW5pc2NyZWVuLnB5) | `94.11% <0.00%> (+29.41%)` | :arrow_up: | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pi-top). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pi-top)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert when merging 58ba4919dc03b3fa67987a6747314dd67d187992 into 19a2b02a4e0881ae450d641c7351b19cb432ae55 - view on LGTM.com

new alerts:

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine :gear: that powers LGTM.com. For more information, please check out our post on the GitHub blog.