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 Lanczos solver #1050

Merged
merged 6 commits into from Oct 18, 2019

Conversation

@NanmiaoWu
Copy link
Collaborator

NanmiaoWu commented Oct 10, 2019

Change get lin_solver_map_ul to make it more general. Originally, it only support string type; now it also support int64_t type.

@@ -552,7 +553,7 @@ namespace phylanx { namespace execution_tree { namespace primitives
"the linear_solver_operation primitive "
"requires "
"that first operand to be a matrix and "
"the second "
"the secotond "

This comment has been minimized.

Copy link
@hkaiser

hkaiser Oct 14, 2019

Member

This was most likely an accidental change.

This comment has been minimized.

Copy link
@NanmiaoWu

NanmiaoWu Oct 16, 2019

Author Collaborator

Yes. I will change it.

@NanmiaoWu NanmiaoWu requested review from hkaiser and scheherzade Oct 18, 2019
Copy link
Member

hkaiser left a comment

Thanks!

@hkaiser hkaiser merged commit 8140853 into master Oct 18, 2019
2 checks passed
2 checks passed
Codacy/PR Quality Review Up to standards. A positive pull request.
Details
build_and_test_phylanx Workflow: build_and_test_phylanx
Details
@hkaiser hkaiser deleted the Lanczos branch Oct 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.