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

FAST Corner Detection with DATAFLOW #127

Closed
norabarlow opened this issue Apr 18, 2022 · 1 comment
Closed

FAST Corner Detection with DATAFLOW #127

norabarlow opened this issue Apr 18, 2022 · 1 comment

Comments

@norabarlow
Copy link

Co-simulation does not finish for the FAST corner detection kernel if I don't use the DATAFLOW pragma.

Does/should xf::cv::fast work without the DATAFLOW pragma?
vt-lib-support pushed a commit that referenced this issue Apr 27, 2022
Squashed 'hpc' changes from f7d1abc..47bdd5f (#574)
49e5d3b 
47bdd5f Merge pull request #127 from changg/wa_for_u280
d98ffc1 fix utils.mk
408860d Merge pull request #126 from changg/wa_for_u280
26fa6b4 wa for u280
50b4c4a Merge pull request #125 from changg/fix_metajson
36dba73 revert the file
51966db fix metajson
2c3e837 Merge pull request #124 from changg/round2-mks
1a6d30e fix mks

Co-authored-by: sdausr <sdausr@xilinx.com>
vt-lib-support pushed a commit that referenced this issue Apr 27, 2022
Squashed 'security' changes from 9ba8ba0..2079b9b (#576)
88205d1 
2079b9b Ethash SC prototype (#150)
c08ada1 merge tutorial (#157)
04302db Merge pull request #155 from changg/fix_issues
f792a67 revert the change
98ccdea test
8fa693a fix security issues
f60f3c8 Update Makefile (#152)
9612886 Fix cr 1114888 1114873 (#151)
e622384 update rc4 benchmark case, reduce to 1 kernel (#149)
8b43251 remove email from Jenkinsfile:https://jira.xilinx.com/browse/CR-1124831 (#145)
3d86755 Fix cr 1114880 1114888 (#146)
49d5003 Merge pull request #144 from liyuanz/replace_cflags
43651c8 replace cflags with clflags
ee65b70 Add open file failure guard and mode for read-only (#143)
a05e213 Merge pull request #141 from liyuanz/replace_blacklist
9095130 replace whiltelist/blacklist to allowlist/blocklist
fb7dfd6 Merge pull request #137 from leol/fix-CR
4454482 Merge pull request #138 from liyuanz/next
ded8552 add mem for mem limit case
80dddf9 Fix for L1 CBC SC benchmarks host build issue
dd68b83 Clang format crc32c_sc.cpp with 3.9.0 version
443fa59 Merge pull request #132 from liyuanz/next
34bc727 Merge pull request #134 from liyuanz/replace_targets
be250a9 update targes
3e28567 update
e0cbc3d Refine crc32c to improve the performance of the resdu line (#133)
884b4f9 update
c790abe update
0256aac update
61cb658 update Makefile and utils.mk
359c5b6 Clang format crc32c_sc.cpp for both 3.9.0 and 8.0.0 (#131)
92cf068 SC L3 design of CRC32C (#123)
c00352b update Makefile for Vitis Flow testcase (#128)
33c3c23 remove ssl linkage from library,json (#127)
c67da48 Merge pull request #126 from changg/metadata
080686b draft metadata files
e3efd4d add aws support (#124)
87f65b0 change 2021.2_stable_latest to 2022.1_stable_latest

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

Hi @norabarlow ,

Modifying the kernel may cause the function (in this case removing the DATAFLOW) to behave abnormally.

vt-lib-support pushed a commit that referenced this issue May 18, 2023
Squashed 'sparse' changes from e8b8ee0..70ab538 (#818)
49cce28 
70ab538 Merge pull request #127 from RepoOps/rep_8
75da629 Merge pull request #128 from changg/add_mem
9a52956 add mem
445938e Replace DEVICE with PLATFORM

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
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@norabarlow @vt-lib-support