-
Notifications
You must be signed in to change notification settings - Fork 671
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
FPGA: Add new sample SVD
#2320
base: development
Are you sure you want to change the base?
FPGA: Add new sample SVD
#2320
Conversation
Adding @pmpeter1 and @tomlenth per request by @KevinUTAT |
DirectProgramming/C++SYCL_FPGA/ReferenceDesigns/svd/src/golden_pca.hpp
Outdated
Show resolved
Hide resolved
DirectProgramming/C++SYCL_FPGA/ReferenceDesigns/svd/src/memory_transfers.hpp
Show resolved
Hide resolved
DirectProgramming/C++SYCL_FPGA/ReferenceDesigns/svd/src/print_matrix.hpp
Show resolved
Hide resolved
DirectProgramming/C++SYCL_FPGA/ReferenceDesigns/svd/src/non_std_covariance.hpp
Outdated
Show resolved
Hide resolved
DirectProgramming/C++SYCL_FPGA/ReferenceDesigns/svd/src/svd_testbench_tool.hpp
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks pretty solid. thx!
DirectProgramming/C++SYCL_FPGA/ReferenceDesigns/svd/src/non_std_covariance.hpp
Outdated
Show resolved
Hide resolved
DirectProgramming/C++SYCL_FPGA/ReferenceDesigns/svd/src/print_matrix.hpp
Show resolved
Hide resolved
DirectProgramming/C++SYCL_FPGA/ReferenceDesigns/svd/src/svd_testbench_tool.hpp
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you get the chance to compute estimated performance to compare with what you obtained in HW?
Example Output when running on the **Terasic DE10-Agilex Development Board**. | ||
|
||
``` | ||
Running on device: de10_agilex : Agilex Reference Platform (aclde10_agilex0) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Run this sample on N6001 to update the readme from DE10 to N6001 as we moved to this board.
|
||
*/ | ||
template <typename T> | ||
class GoldenPCA { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PCA? I guess you meant SVD? Or maybe you didn't mean to add this file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I copy the golden PCA to obtain the eigen values for reference, so it technically is PCA
Description
Adding a new FPGA reference design Singular Value Decomposition.
Testing
Running reg-tests, will update test links once finished.
Checklist
Administrative
Code Development
Security and Legal
Review