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 notes for the missed step in linux compiling #413

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

Conversation

bigbearishappy
Copy link

The compile step in the ncnn documents don't tell us to make install it after compilation finished.
Nor the document in the demo_ncnn.
So add the note here.

@codecov
Copy link

codecov bot commented Mar 30, 2022

Codecov Report

Merging #413 (80f3ded) into main (3cda376) will decrease coverage by 0.04%.
The diff coverage is n/a.

❗ Current head 80f3ded differs from pull request most recent head 8cfd387. Consider uploading reports for the commit 8cfd387 to get more accurate results

@@            Coverage Diff             @@
##             main     #413      +/-   ##
==========================================
- Coverage   74.25%   74.20%   -0.05%     
==========================================
  Files          66       66              
  Lines        4459     4459              
  Branches      756      756              
==========================================
- Hits         3311     3309       -2     
- Misses        969      970       +1     
- Partials      179      180       +1     
Flag Coverage Δ
unittests 74.20% <ø> (-0.05%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
nanodet/data/transform/warp.py 84.48% <0.00%> (-1.15%) ⬇️

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

Hardware:
1 raspberry pi zero 2
2 camera:Raspberry Pi Camera Rev 1.3
3 LCD:4inch RPi LCD(A) waveshare SpotPear

dtoverlay config:
dtoverlay=fbtft,spi0-0,piscreen,reset_pin=25,dc_pin=24
dtoverlay=ads7846,penirq=17

PS:remeber to set the LCD spi speed up to 30MHz
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant