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

Rewrite #98

Merged
merged 39 commits into from
Feb 22, 2018
Merged

Rewrite #98

merged 39 commits into from
Feb 22, 2018

Conversation

ranjanan
Copy link
Member

@ranjanan ranjanan commented Feb 18, 2018 •
edited
Loading

This PR improves Circuitscape in the following ways:

  1. Improves structure and code organization
  2. Gets rid of type instabilities.
  3. Adds a stable CHOLMOD solver path.
  4. Adds parallelism (for writing) on the CHOLMOD solver path.

Fixes #99 , #96 , #55

@ranjanan
Start rewrite
cccb885
@ranjanan
First network case works
86fb49c 
We've also padded user points
@ranjanan
Fix calculation in total number of solves
9d1e540
@ranjanan
Fix bug in calculation of total no of solves
1511ddf
@ranjanan
Try to make single precision work
e4fc507
@ranjanan
Make tolerance single precision too, for this to work
ddceb3a
@ranjanan
First crack at cholmod path, single precision fixes
7069c21
@ranjanan
Reorg file writing stage
56524c8
@ranjanan
Apply to amg solver path too
0a441ee
@ranjanan
Throw appropriate warning at single precision chol
b77acf5
@ranjanan
Add logging with file writing
063a38c
@ranjanan
Add cholmod solver mode with parallel file writing
8545f6c
@ranjanan
Get simplest raster pairwise working
983ea7b
@ranjanan
Support a few more config files - raster pairwise
f36943a
@ranjanan
Fix bug in point file polygon scenario
cbcf909
@ranjanan
Some cleanup
8951d9a
@ranjanan
Get basic advanced mode raster working
7074bf1
@ranjanan
Add sgVerify10_resistances back (deleted by mistake)
11a1aa6
@ranjanan
Get network advanced mode to work
75d6cbc
@ranjanan
Advanced code - network mode
a6a6d42
@ranjanan
First crack at onetoall support
f1fce16
@ranjanan
First onetoall works
97753cd
@ranjanan
Add unique points to the final result
d6d1706
@ranjanan
Both one to all and all to one modes run
2189011
@ranjanan
Make ini tests work except shortcut
559e935
@ranjanan
resistance shortcut passed
fcee171
@ranjanan
Fix type instabilites
0208a5b
@ranjanan
Get rid of some prints
0fab507
@ranjanan
Remove comments and dead code
74ea0ed
@ranjanan
Memory improvements on some current map calculation
fd94e9a
@ranjanan ranjanan merged commit a72dad6 into master Feb 22, 2018
@ranjanan ranjanan deleted the reqrite branch February 22, 2018 07:08
This was referenced Feb 22, 2018
Closed
Closed
@amatem amatem mentioned this pull request Feb 28, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ranjanan