-
Notifications
You must be signed in to change notification settings - Fork 142
xf::remap() function optimization (URAM utilization) #5
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
Open
alekrop
wants to merge
81
commits into
Xilinx:dev-uram
Choose a base branch
from
ZoTechGroup:master
base: dev-uram
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…in c-sim and co-sim. II=1.
Updated Remap, Minmax, StereoBM, MST, Delay and Dense-non-pyramidal o…
…ter on the board.
…thesis of URAM reading (checked via board implementation).
… and become incompatible with kernel)
…lication with kernel.
…oh rates is done.
…OLL for cleanup loop with fully partioned arrays.
…ples to call top level function of Gaussian filter from xfopencv library
# Conflicts: # .gitignore # include/imgproc/xf_remap.hpp
…s bm_state member as separate integers. To reduce number of kernel parameter only "flexible" parameter (which are not calculating from template) are passing. The XOCC flow can't work with structures/class as kernel parameter. Unfortunately if try to pass structure as array of simple type (char/int/etc) then impossible copy element of this array to members of the kernel local instantiation of structure - compilation passing, sw emulation passing, but hw emulation and FPGA run fail due to XOCC issue.
…tion and FPGA (kernel suspend)
…ly done (no co-sim and other checks).
…sted fix of possible padding problem.
…etics) are rolled-back.
… template parameters are removed.
…nction is roughly done (no co-sim and further checks).
…in auMedianBlur() function by restructuring loops.
… in auMedianBlur() to exclude deadlocks in co-sim.
…kariound to exclude X values in co-sim.
…e xFPyrDownGaussianBlur() to decrease its pipeline depth.
…PyrDownGaussianBlur() function.
… every partition equal to URAM to overcome cascading timing issue.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
URAM usage is enabled by template parameter USE_URAM, which is false by default. Functionality is bit-exact with BRAM implementaion: checks pass C-sim, RTL co-sim and zcu104_rv board runs for NN and Bi-linear filtering modes on fractional coordinates.