I'm trying to implement an algorithm for prediction of missing nodes using Matlab. But, there are some errors in my coding , Could anyone please correct my code?

Question

SUNANNA S S 2017-3-30

0
链接

此问题的直接链接

https://ww2.mathworks.cn/matlabcentral/answers/332683-i-m-trying-to-implement-an-algorithm-for-prediction-of-missing-nodes-using-matlab-but-there-are-so

评论： SUNANNA S S 2017-3-30

clc;
edges = dlmread('twitnet.csv');
edges(all(edges == 0, 2), :) = [];  
[uedges, ~, erow] = unique(edges.', 'stable');  %transpose and stable to give the same output as graph
X = full(sparse(erow(1:2:end), erow(2:2:end), 1, numel(uedges), numel(uedges)));
X = X + X.';
CDcalculation(CD); %consistency degree 
* error after this *

Thanks in advance

0 个评论
显示 -2更早的评论隐藏 -2更早的评论

请先登录，再进行评论。

请先登录，再回答此问题。

Answer 1

Walter Roberson 2017-3-30

0
链接

此回答的直接链接

https://ww2.mathworks.cn/matlabcentral/answers/332683-i-m-trying-to-implement-an-algorithm-for-prediction-of-missing-nodes-using-matlab-but-there-are-so#answer_260985

在 MATLAB Online 中打开

Consider your line

[j,k]=CN;

You are within a "for j" loop, so you would be overwriting the loop control variable "j". Overwriting a loop control variable is almost always a mistake.

You are writing to multiple outputs. The only way that could be valid would be if CN were a function (whose code you do not show.) That would be consistent with the fact that you have not defined CN at that point.

But on the next line you have

CN=find(X(j,1)==X(j,k)); %consistency calculation

which assigns to CN as a variable. If you managed to execute that line (because you had a function named CN with multiple outputs) then your next loop through would have to fail, when CN had become a numeric variable, as it is not possible to assign a numeric variable to multiple outputs.

2 个评论
显示无隐藏无

SUNANNA S S 2017-3-30

Thank you for your reply Sir. In this, actually I want to calculate CN(j,k) = find(X(i,j)==X(i,k)), but I don't know how to do it in matlab. Actually,CN is the consistency of jth column with all other k columns. So, I'm fully confused about how to do it. Please help.

SUNANNA S S 2017-3-30

编辑：Walter Roberson 2017-3-30

在 MATLAB Online 中打开

As per your guidance, I calculate consistency(CN) and consistency degree(CD) as separate functions, but still errors remain.Please check my code Sir.

function CNcalculation()
for j=X(:,1): X(:,:)
    for i=1:length(X)
        if find(X(i,j)==0);
        for k=X(:,2):X(:,:)
            CN=find(X(i,j)==X(i,k));
        end
        end
    end
end
CNcalculation(CN); 
%for CD calculation
function CDcalculation()
CNcalculation(CN);%consistency calculation 
for j=X(:,1): X(:,:)
U(j)=X(sum(:,X == 1) ); %no: of linked nodes of jth column
CD=max(CN/U(j));%consistency degree calculation
end
CDcalculation(CD);

请先登录，再进行评论。

Answer 2

Guillaume 2017-3-30

0
链接

此回答的直接链接

https://ww2.mathworks.cn/matlabcentral/answers/332683-i-m-trying-to-implement-an-algorithm-for-prediction-of-missing-nodes-using-matlab-but-there-are-so#answer_261009

在 MATLAB Online 中打开

Note: the only part of your code that works is actually the one I wrote in answer. After that, nothing makes sense. In addition to the issues that Walter pointed out we start with:

for j = X(:,1) : X(:, :)

I have absolutely no idea what you're meaning to do with that line, but it's certainly not going to do what you want. You're passing vectors and matrices as the boundaries of the loop, matlab ignores these and only use the first element. So that line is equivalent to:

for j = X(1,1):X(1,1)

So the j loop will only have one step.

Similarly, your k loop is equivalent to:

for k = X(1,2):X(1,1)

Now since X is an adjacency matrix with only 0 and 1, that loop will either never execute or execute once. Hardly worth a loop.

Then we have the [i,j]=CN that Walter mentioned. That also makes no sense.

I have no idea what you're trying to do, but you should seriously consider learning more about matlab syntax before proceeding.

1 个评论
显示 -1更早的评论隐藏 -1更早的评论

SUNANNA S S 2017-3-30

Sir,actually I want to find the consistency of each columns with all other columns. So that I want to assign j pointer to first column and k to all other columns, that's why I use this loop,but I cannot achieve the result that I expected,as you said. I don't know how to do it and I had tried many ways by seeing some tutorials,but that's not I want. Please help.

请先登录，再进行评论。