Open
Conversation
* generate affinity matrix from geometries * refactor and document, and sensitivity * sensitivity and other movment modes * renames * return missing methods * fix RW algorithm * reorganise documentation for new package structure * more docs * jet * docs and examples * tests * passing tests * bugfix jet * docs and tests * reorganise files * move moving files * quarto docs * working notebooks * notebook fixes * more notebook work * ConnectedHabitat -> FunctionalHabitat * renames * reorganise for finalize step * add brams calibration step * bugfixes * ConScapeProblem * transition -> step * passing tests * no OldConScape in extras * no threading in LinearSolver * return OldConScape to extras * bugfixes * Optim in docs * use Graphs from branch until its merged * move types out of ConScape.jl fiel * remove init * use the ReadOnlyArrays package for ReadOnlyArray * fix mosaic tests * move sources block down * run CI on any PR * no lts in CI for now * remove JET its failing in CI * mosaic is slightly broken * bugfix windowed tests * remove unreliable test
Member
Author
|
We should also test if changes in theta increase the similarity as expected. Another possibility is to try Float128 for better accuracity solves. |
Merged
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
This PR tests the correlation between LeastCostPath and RSP with high theta, and RandomWalk/RSP with low theta.
@ninbrm this will need your input as to whether this makes sense, but it seems like a fairly intuitive way to explicitly link and test the three approaches.
The betweenness measures dont end up exactly the same. The correlation increasese as theta gets lower/higher but the overall magnitudes dont necessarily match. Thats why
coris used rather thanisapprox.On the LCP / RSP high theta side things start to break with NaNs and other kinds of overflow as theta gets too high, so we can't test all the outputs.
Another consideration is we should test it accross distance_transformation values, like
alpha=2.0, as the correlation seems to drop off a lot.