-
Notifications
You must be signed in to change notification settings - Fork 6
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
Priority sites criteria expecting different structure for strong_pred
than what is implemented
#633
Comments
Hi @Rosejoycrocker, there are a few things about this issue I don't understand, would you mind helping me by answering a few questions?
About the connectivity:
|
Hi @Zapiano ,
We can chat about this if that's easier :) |
strong_pred
is currently a vector which at indexi
, gives the id of the site which is connected to sitei
and provides the greatest number of incoming connections to sitei
. Previously (inADRIA_repo
, the Matlab version),strong_pred
was annsites
by 2 matrix, with the first column being site ids:https://github.com/open-AIMS/ADRIA_repo/blob/bc29474555015b88fa0a10f7f935408f0b4f0e76/ADRIAfunctions/siteConnectivity.m#L196-L197
This structure is then used in the MCDA priority predecessors criteria to give higher value to sites which are the strongest predecessors of priority sites:
https://github.com/open-AIMS/ADRIA_repo/blob/bc29474555015b88fa0a10f7f935408f0b4f0e76/ADRIAfunctions/ADRIA_DMCDA.m#L66-L70
This structure is similarly used in the current version of ADRIA:
ADRIA.jl/src/decision/dMCDA.jl
Line 616 in c07dc5f
But instead of
predec[:,1]
being site ids, it is a copy ofstrong_pred
, which is a vector. To fix this,predec[:,1]
as used in the above, should be swapped out for the set of siteids, or justpriority_sites
should be used to index (assuming it is ordered and of integer type).The text was updated successfully, but these errors were encountered: