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.

Sign up for GitHub

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

Jump to bottom

Project architecture upgrade #250

Open
wants to merge 36 commits into
base: main
Choose a base branch
from
Open

Project architecture upgrade #250

wants to merge 36 commits into from
+2,527 −2,433

Conversation

HaoboGu
Copy link
Owner

@HaoboGu HaoboGu commented Jan 17, 2025

Closes #229

HaoboGu added 15 commits January 6, 2025 23:50
@HaoboGu
wip: architecture upgrade
212d821 
Signed-off-by: Haobo Gu <[email protected]>
@HaoboGu
feat(reporter): finalize reporter trait
012192a 
Signed-off-by: Haobo Gu <[email protected]>
@HaoboGu
wip: working usb reporter
d532d6d 
Signed-off-by: Haobo Gu <[email protected]>
@HaoboGu
feat: add Runnable trait, add channel size to input device, process…
3e69677 
…or, reporter and listener trait

Signed-off-by: Haobo Gu <[email protected]>
@HaoboGu
wip: add vial reporter and listener
be9c32a 
Signed-off-by: Haobo Gu <[email protected]>
@HaoboGu
wip: move trait to hid mod, refine trait again
4b357ce 
Signed-off-by: Haobo Gu <[email protected]>
@HaoboGu
feat: use new traits
1ce3a1c 
Signed-off-by: Haobo Gu <[email protected]>
@HaoboGu
Merge remote-tracking branch 'origin/main' into feat/architecture_ref…
e54508d 
…actor

Signed-off-by: Haobo Gu <[email protected]>
@HaoboGu
fix: fix formatting errors after merging main branch
82aecd7 
Signed-off-by: Haobo Gu <[email protected]>
@HaoboGu
Merge remote-tracking branch 'origin/main' into feat/architecture_ref…
950603b 
…actor
@HaoboGu
refactor: use same run_ble_keyboard, simplify code
5937705 
Signed-off-by: Haobo Gu <[email protected]>
@HaoboGu
feat: add light service back
000bfae 
Signed-off-by: Haobo Gu <[email protected]>
@HaoboGu
Merge branch 'main' into feat/architecture_refactor
d69ab35
@HaoboGu
feat(ble): add battery service back
9817ef7 
Signed-off-by: Haobo Gu <[email protected]>
@HaoboGu
Merge branch 'main' into feat/architecture_refactor
008e495
@github-actions GitHub Actions

This comment was marked as outdated.

Sign in to view
@github-actions GitHub Actions

This comment was marked as outdated.

Sign in to view
@github-actions GitHub Actions

This comment was marked as outdated.

Sign in to view
Repository owner deleted a comment from github-actions bot Jan 19, 2025
@github-actions GitHub Actions

This comment was marked as outdated.

Sign in to view
@github-actions GitHub Actions

This comment was marked as outdated.

Sign in to view
@github-actions GitHub Actions

This comment was marked as outdated.

Sign in to view
@github-actions GitHub Actions

This comment was marked as outdated.

Sign in to view
@github-actions GitHub Actions

This comment was marked as outdated.

Sign in to view
@github-actions GitHub Actions

This comment was marked as outdated.

Sign in to view
@HaoboGu HaoboGu changed the title WIP: Project architecture upgrade Project architecture upgrade Jan 23, 2025
@HaoboGu
refactor: clean code
34ada55 
Signed-off-by: Haobo Gu <[email protected]>
@github-actions GitHub Actions

This comment was marked as outdated.

Sign in to view
@github-actions GitHub Actions

This comment was marked as outdated.

Sign in to view
@github-actions GitHub Actions

This comment was marked as outdated.

Sign in to view
@github-actions GitHub Actions

This comment was marked as outdated.

Sign in to view
@github-actions GitHub Actions
Copy link

PR build size

  text	   data	    bss	    dec	    hex	filename
129804	     84	  41096	 170984	  29be8	rmk-nrf52840

Diff

    FILE SIZE        VM SIZE    
 --------------  -------------- 
  +4.0% +4.21Ki  +4.0% +4.21Ki    .text
  +1.9% +3.80Ki  [ = ]       0    .debug_loc
  +1.8% +2.44Ki  [ = ]       0    .debug_line
  +1.0%    +400  [ = ]       0    .symtab
  +1.2%    +212  [ = ]       0    .debug_frame
  +0.2%     +14  [ = ]       0    .debug_abbrev
   +12%      +5  [ = ]       0    [Unmapped]
  -3.0%     -14  [ = ]       0    .defmt
  -0.2%     -32  [ = ]       0    .debug_aranges
  -0.4%     -64  -0.4%     -64    .rodata
  [ = ]       0  -0.5%    -212    .bss
  -1.0% -1.27Ki  [ = ]       0    .strtab
  -0.3% -2.49Ki  [ = ]       0    .debug_info
  -2.6% -2.62Ki  [ = ]       0    .debug_ranges
  -2.6% -32.8Ki  [ = ]       0    .debug_str
  -1.0% -28.3Ki  +2.4% +3.94Ki    TOTAL

This was referenced Jan 24, 2025
Draft
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Architecture upgrade, make input device - input processor - reporter decoupled with each other
1 participant
@HaoboGu