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

Demosaicing Makefile fails with check_opencv #98

Closed
nlbutts opened this issue Dec 12, 2021 · 2 comments
Closed

Demosaicing Makefile fails with check_opencv #98

nlbutts opened this issue Dec 12, 2021 · 2 comments

Comments

@nlbutts
Copy link

nlbutts commented Dec 12, 2021

I followed the getting started guide: https://github.com/Xilinx/Vitis_Libraries/blob/master/vision/docs/getting-started-with-hls.rst
When I try to make:

make run XPART='xcu200-fsgd2104-2-e' CSIM=1

This generates the following message:

nlbutts@Dell-G15-5515:~/projects/xilinx/Vitis_Libraries/vision/L1/examples/demosaicing$ make run XPART='xcu200-fsgd2104-2-e' CSIM=1
XPART is directly set to xcu200-fsgd2104-2-e
make: *** No rule to make target 'check_opencv', needed by 'setup'.  Stop.

I see the check rule in the Makefile, but I can't figure out why it isn't finding the rule.

@vt-lib-support
Copy link
Collaborator

Hello @nlbutts ,

Please try to use the below command to run CSIM, meanwhile we check on the issue:

make run CSIM=1 DEVICE=<path-to-platform-directory>/< platform >.xpfm

vt-lib-support pushed a commit that referenced this issue Apr 27, 2022
b8aa0ac Merge pull request #98 from changg/round2-mk
0dafa9e fix
69ce1de Merge pull request #97 from changg/round2-mk
33635fb fix utils.mk
bb09f85 Merge pull request #96 from tuol/fix_cr_1125397
26bfd12 upate description.json for GUI
6142bf4 Merge pull request #95 from tuol/fix_cr1110852
18b6b69 add description for cscmvSingleHBM
489d6fe Merge pull request #94 from changg/wa_u280
c83ed11 WA for u280
70d666c Revert "wa for xilinx_u280_xdma_201920_3"
27306c5 Merge pull request #93 from changg/wa_u280_2019
2b4ec88 wa for xilinx_u280_xdma_201920_3
af1d094 Merge pull request #92 from liyuanz/replace_cflags
a9ff957 replace cflags with clflags
eb58a83 Merge pull request #91 from tuol/fix_cr_1110852
c9ca9ab add description to L2/tests/cscmvSingleHbm
b8e0210 Merge pull request #90 from tuol/fix_cr_1083211
75204c8 remove 'exclude' from description.json
965b2e0 Merge pull request #88 from liyuanz/replace_blacklist
0e87eee replace whiltelist/blacklist to allowlist/blocklist
ad26de7 Merge pull request #87 from liyuanz/next
3db258c increase time
def14fe Merge pull request #85 from changg/add_extraflags
f8a6122 fix makefile
031de2c Merge pull request #84 from liyuanz/next
f10953c increase time
4745f24 Merge pull request #83 from changg/fix_utils
6fb2652 fix utisl
7d9278f Merge pull request #82 from liyuanz/replace_targets
c1c2de5 update targes
68ae52e Merge pull request #80 from changg/metadata
6225d51 draft metadata files
e74cf2b change 2021.2_stable_latest to 2022.1_stable_latest

Co-authored-by: sdausr <sdausr@xilinx.com>
vt-lib-support pushed a commit that referenced this issue Apr 27, 2022
f7d1abc Merge pull request #122 from tuol/disable_2_case
ae62691 disable 2 case due to U250 platform change
3af143e Merge pull request #118 from tuol/fix_cr_1122542
3e7f919 temporally disable L3/tests/mlp, due to U250 platform change
1728d13 update opts.cfg
98d3f3f Merge pull request #117 from yuanqian/next
8639708 remove email from Jenkinsfile:https://jira.xilinx.com/browse/CR-1124831
18a7458 Merge pull request #116 from changg/wa_u280_201920
86e28ef WA for xilinx_u280_xdma_201920_3
07abe54 Merge pull request #114 from liyuanz/replace_cflags
7cb157c replace cflags with clflags
0196ded Merge pull request #113 from changg/cov_fix
fc100b4 cov fix
b201f43 cov fix
14067e6 Merge pull request #110 from liyuanz/next
bbe42e9 fix bug
257677d Merge pull request #109 from changg/pr_108
79db50c fix makefiles
984a71c update Makefile and utils
daf9820 Merge pull request #106 from liyuanz/replace_blacklist
28fe2ed replace whiltelist/blacklist to allowlist/blocklist
981b5a2 Merge pull request #105 from changg/pr_104
2f45a63 add time for hw_build
a21b8db add time
7256e35 add time
5f2c36a Merge pull request #102 from changg/add_extraflags
acce305 fix utils.mk
74536af fix utils.mk
3c0647e Merge pull request #101 from liyuanz/next
fc26744 increase mem
7a1b220 Merge pull request #99 from changg/fix_mks
055c521 fix typ
44ff7b9 fix utils.mk
4050d17 Merge pull request #98 from liyuanz/replace_targets
b0157d6 update targes
e41fc60 Merge pull request #96 from changg/metadata
f6d1e26 draft metadata
0bbb982 change 2021.2_stable_latest to 2022.1_stable_latest

Co-authored-by: sdausr <sdausr@xilinx.com>
@vt-lib-support
Copy link
Collaborator

This has been fixed in the latest release. Please reopen if needed.

vt-lib-support pushed a commit that referenced this issue Oct 19, 2023
d5c1b1c Merge pull request #165 from yuanqian/fix_api_meta-2
0d165fe update
3120f1e add visble and target_domain
32f9bf9 [fix bug]1.fix abs path in makefile of imagepoint 2. api.json of L2 visible=false
3cc0a3f [doc]add L2 features and tutorial
90a6c05 [doc]leagal
6bbe7d5 Merge pull request #163 from yunleiz/next
1c9d082 [doc] refine tutorial
4f7d0c1 [format] L3
2a9463b Merge branch 'next' of https://gitenterprise.xilinx.com/FaaSApps/xf_ultrasound into next
b67854f [doc]features/details_L2.rst add delay sample mult scanline
cc64796 [format] L3
858df2d [L3] refine accuracy
beb874b Merge pull request #160 from congt/next
cfae80f Merge pull request #162 from yunleiz/next
ddfcd7b [L3] remove small test
9c8b25c Merge pull request #161 from yunleiz/next
cec21dd [L3] refine makefile
8e5fff5 Merge branch 'next' of gitenterprise.xilinx.com:FaaSApps/xf_ultrasound into next
0cff04d [L2]add description for graph_imagepoints and graph_focusing
293dad7 [L2] fixed bugs
a0d89b4 [L2]correct description.json for L2 graph_scanline
9a8a73f [L2] fixed a makefile bug
df8f760 [L2]description.json for L2 mult delay sample and scanline
72b3403 [L2]add graph_delay.json
ef3dc2f [L2 tutorial]
abef6b4 [meta data]
331686f Merge pull request #159 from yunleiz/next
454ffab [remove] data2png move to models
ceff96b [doc] L3 doc update
0856fa7 Merge pull request #158 from yunleiz/next
d94c36e [L3] refine meta data and description
c322805 [L3] update meta and format json.hpp
df5048e [refine]L2 imagepoint remove hardnumber
0d5169f [L1 legal]add header for us_example_parameter
daf6f6d [L3] add license
c25e8e1 [L3] refine makefile
aa28f20 [L3] refine makefile
234ce6e [L3] refine api.json
67b4d83 Merge pull request #157 from yunleiz/next
4a2592c [model] format
b6bff22 [L3] scanline AllinAIE pl freq change to 400
a55e4d7 Merge pull request #152 from changg/aie_cli
085fdc4 Merge pull request #156 from yunleiz/next
db9a639 Merge branch 'next' into aie_cli
0fa1b41 [L1 L3] move dev branch
46a1491 Merge pull request #155 from yunleiz/next
0d9fdae [L3] move models from dev branch
38df5eb [L3] move models from dev branch
346a48e Merge pull request #154 from congt/next
af30489 [L3] move from dev branch
4225747 [L2]add data/ for graph_scanline
b0ce2b3 resolve conflict
8e4cd3a resolve conflict
4196533 resolve conflict
f672081 [standlization]
3263251 fix
f65de04 ad fix
189e34c format json, and add config file
2ee80ac Revert "convert to config"
2b5a833 convert to config
11298bc port to unified command for aie apis
1724ce1 Merge pull request #147 from yunleiz/next
b2aa8b0 [clean] merge mult
e02354e Merge branch 'next' of gitenterprise.xilinx.com:FaaSApps/xf_ultrasound into next
e7cfa29 Merge pull request #150 from congt/next
c934c58 [models] format
1d3e3ab [L3] add scanline_AllinAIE
8d64da9 [add file]L2/include delay sample and mult
cb8bf65 Merge branch 'next' of https://gitenterprise.xilinx.com/FaaSApps/xf_ultrasound into next
c542d5f [standlization]clang-format and legal of makefile
9af2c9f [add L2] graph imagepoint, focusing, scanline
459d38c Merge pull request #148 from congt/next
ff73213 remove unused files
d52f71b [update] interpolation
1968881 [format] apodi and interp
b868bee [E2E]fix coding style of L2/tests/graph_delay/aie_graph/graph.cpp
10fd877 [E2E]add L1
4aafd8f [refine] api.json
9c1d672 [standlization]add L1 L2 of delay sample mult
c7cb72a [api.json] refine api.json
ad3e52d [E2E] add apodization and interpolation unit test
41f76ab Merge pull request #104 from liyuanz/revert-103-next
2f3827b Revert "set visible=False in api.json"
9c88301 Merge pull request #103 from liyuanz/next
3daf775 update
741f03e update
a5bb4e3 Merge pull request #98 from changg/fix_target_domain
211e922 update
a545cf3 fix target domain
42480c3 change 2023.1_stable_latest to 2023.2_stable_latest
f925024 Merge pull request #95 from changg/fix_copyright
d0bf220 fix copyright

Co-authored-by: sdausr <sdausr@xilinx.com>
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

No branches or pull requests

2 participants