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

Add led effect to DCSpy #39

Open
wants to merge 95 commits into
base: master
Choose a base branch
from
Open

Add led effect to DCSpy #39

wants to merge 95 commits into from

Conversation

emcek
Copy link
Owner

@emcek emcek commented Nov 2, 2021

Description

Allow to pulse led when i.e. master caution is set.

what when 2 filed request effect? Flow

  • apply effect_1
  • apply effect_2
    • shutdown
    • save effect_1
    • apply effect_2

optional flow 1

  • deactive effect_1
    • only remove from heap

optional flow 2

  • deactive effect_2
    • remove from heap effect_2
    • shutdown
    • apply effect_1

add callback to exiting BIOS_VALUE

BIOS_VALUE = TypedDict('BIOS_VALUE', {'class': str, 'args': PROTO_ARGS, 'value': Union[int, str], 'max_value': int, 'callback': str}, total=False)

led setting

where (if at all) set options for LED effect

  • config.yaml
  • hardcode
  • BIOS_VALUE (old) as dict

test start_led_pulse vs. start_led_flash

try parametrize setting in config if you end-up with one field

findings

  • No thread, no events, no sleep time needed
  • need keep track of active effects in OrderedDict (due to py3.6 can not be regular dict)
  • when one effect is active others can not be applied

Todo

  • Tests
  • Documentation

@emcek emcek added this to the Release 1.7.0 milestone Nov 2, 2021
@emcek emcek self-assigned this Nov 2, 2021
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@emcek emcek added the enhancement New feature or request label Dec 30, 2021
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@emcek emcek removed this from the Release 1.7.0 milestone Jun 7, 2022
# Conflicts:
#	CONTRIBUTING.md
#	dcspy/aircraft.py
#	dcspy/utils.py
#	resources/config.yaml
#	tests/test_aircraft.py
#	tests/test_bios.py
#	tests/test_led_sdk.py
#	tests/test_logitech.py
#	tests/test_utils.py
#	uml/classes.puml
# Conflicts:
#	CONTRIBUTING.md
#	dcspy/aircraft.py
#	dcspy/utils.py
#	resources/config.yaml
#	tests/test_aircraft.py
#	tests/test_bios.py
#	tests/test_led_sdk.py
#	tests/test_logitech.py
#	tests/test_utils.py
#	uml/classes.puml
@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 1, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

emcek and others added 3 commits January 27, 2024 17:04
# Conflicts:
#	.pylintrc
#	CONTRIBUTING.md
#	dcspy/aircraft.py
#	dcspy/config.yaml
#	dcspy/logitech.py
#	dcspy/sdk/led_sdk.py
#	dcspy/utils.py
#	tests/test_aircraft.py
#	tests/test_led_sdk.py
#	tests/test_utils.py
#	uml/classes.puml
@emcek emcek added this to the Release 3.2.0 milestone Jan 29, 2024
Copy link

sonarqubecloud bot commented Feb 1, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@emcek emcek removed this from the Release 3.2.0 milestone Mar 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant