-
Notifications
You must be signed in to change notification settings - Fork 2
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
version 1.1.0 #10
Merged
version 1.1.0 #10
Conversation
This file contains 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
Solvers added
libs static added
considering abandon generic gmres solver. Although iluapck is slow, we would definite need it for now.
Update README.md
the license can be found in each files within Ipopt package, however, we only build the exe for our Matlab interface.
For performance of ``exprtk``, it can match almost 80% percent of naive implementation. However, it would take 1~2 minutes on compiling the program. Any way, it is better than MATLAB's dynamical allocation with each step. A workaround is use preallocation as ``std::vector`` does in C++, which uses an aggressive way for appending space.
Tried Makefile with -O flag to boost compiling process with exprtk. It does help a lot, let the time drop from minutes to tens of seconds. Tried to integrate OPENMP with programs piece by piece. This package is designed to utilize shared memory within Matlab's environment, since Matlab has not opened its MPI api yet.
``boundary`` module's syntax changed. Signed-off-by: Yimin <zhongym8903@gmail.com>
There is a potential so-called ``static TLS`` problem, which related to loading libs. Since Matlab is trying to pre-loading some libs on start, it might be a problem if you are going to load another lib later. However, this is for sure a bug from earlier version of Matlab. The problem could be somewhere about OPENMP. Removing ``OpenMP`` is considered. Or using another libomp might help. Signed-off-by: Yimin <zhongym8903@gmail.com>
Now all versions older than 2013a should work. Signed-off-by: Yimin <zhongym8903@gmail.com>
Modified Makefile for building on Linux x86_64 only. This package is for performance testing only now. Signed-off-by: Yimin <zhongym8903@gmail.com>
Signed-off-by: Yimin <zhongym8903@gmail.com>
* Add support for constant input in Matlab. it will reduce a lot of memory during each call. * Add default constructor back to Boundary module. It can be set Dirichlet Boundary condition later, not at very beginning. Signed-off-by: Yimin <zhongym8903@gmail.com>
Removed member b_edges as extra waste of memory. More efficient with dofs member function as a last call to decide which are dofs and which are not. However, compiling process is still slow. Signed-off-by: Yimin <zhongym8903@gmail.com>
An idea is to make set boundary method binded with numbering, but this seems not very necessary and will slow down the performance when there are too many boundaries, and cannot group them easily. ``set_boundary`` is one way to use experssion to group edges into boundaries. Also there is some other way to group edges, like markers, but this may take up some more memories and using Matlab to form this groups maybe too slow, though it will be very fast, and usually one pass only. Signed-off-by: Yimin <zhongym8903@gmail.com>
Signed-off-by: Yimin <zhongym8903@gmail.com>
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.
No description provided.