You need to use \caption
before \label
; otherwise, the string picked for the reference will be the last one where an anchor was set (typically the one of a sectional unit), in this case, the one from \chapter
. Here's a modified version of your code showing the desired result:
\documentclass[11pt, letterpaper]{book}
\renewcommand{\thechapter}{\Roman{chapter}}
\renewcommand{\thesection}{\arabic{section}}
\usepackage[chapter]{algorithm}
\usepackage{algorithmicx}
\usepackage{algpseudocode}
\renewcommand{\thealgorithm}{\arabic{chapter}.\arabic{algorithm}}
\begin{document}
\chapter{First Chapter}
\begin{algorithm}
\begin{algorithmic}[1]
\State Some code here
\end{algorithmic}
\caption{A test algorithm}
\label{alg:algorithm1}
\end{algorithm}
Algorithm~\ref{alg:algorithm1} states blah blah blah and algorithm~\ref{alg:algorithm2} states blah blah blah.
\begin{algorithm}
\begin{algorithmic}[1]
\State Some code here
\end{algorithmic}
\caption{Another test algorithm}
\label{alg:algorithm2}
\end{algorithm}
\end{document}
If you don't want to give explicit captions to your algorithms, you can use the \phantomcaption
command from the caption
package (this, however, could be confusing since you are referencing by number some objects which aren't really numbered):
\documentclass[11pt, letterpaper]{book}
\usepackage{caption}
\renewcommand{\thechapter}{\Roman{chapter}}
\renewcommand{\thesection}{\arabic{section}}
\usepackage[chapter]{algorithm}
\usepackage{algorithmicx}
\usepackage{algpseudocode}
\renewcommand{\thealgorithm}{\arabic{chapter}.\arabic{algorithm}}
\begin{document}
\chapter{First Chapter}
\begin{algorithm}
\begin{algorithmic}[1]
\State Some code here
\end{algorithmic}
\phantomcaption
\label{alg:algorithm1}
\end{algorithm}
Algorithm~\ref{alg:algorithm1} states blah blah blah and algorithm~\ref{alg:algorithm2} states blah blah blah.
\begin{algorithm}
\begin{algorithmic}[1]
\State Some code here
\end{algorithmic}
\phantomcaption
\label{alg:algorithm2}
\end{algorithm}
\end{document}
It is not completely clear to me which algorithm typesetting package are you using, since you did not include the document preamble, and your code is a strange jumble of several different syntaxes, mostly looking like the old algorithms
package syntax, but not quite. Your tags seem to indicate that you are using the algorithmicx
bundle, so I assume that you use the algpseudocode
package. You need to correct some syntax, for example algpseudocode
does noy capitalize command, only first letter, so you should have \If ... \Else ... \EndIf
etc.
Also, command \For
and \If
take an argument, so you need
\If{something something}
The \If
does not use \Then
, you should leave that out.
Every line must have a command, so you should either define your own commands for things like "Given:", or just use \State
.
The following will hopefully do what you expect it to do:
\documentclass{article}
\usepackage{algpseudocode}
\usepackage{algorithm}
\begin{document}
\begin{algorithm}
\caption{Blah blah}
\begin{algorithmic}[1]
\State Given: User-Noun Matrix $X$
\For{$i = 1 \to n$}
\State Let $S_i$ = {$s{i,j}$ | $0\le j \le l_i$} be set of nodes which have links to $i$;
\If{$\left(l_i=0 \hbox{OR} l_i=1 \right)$}
\State $b_i=0$;
\Else
\State Compute the similarity score vectors $r_{s,1},r_{s,2},...,r_{s,l_i}$ for each $S_i=s_1,s_2,...s_{l_i}$; \\
\State Construct $l_i \times l_i$ matrix $R_i= \left[r_{s,1},r_{s,2},...,r_{s,l_i}\right]$;
\State Take the average of all non-diagonal elements in $R_i$ to obtain $\hat{r_s}$;
\State $b_i= 1/\hat{r_s}$;
\EndIf
\EndFor
\State \Return vector $b$ of bridging scores;
\end{algorithmic}
\end{algorithm}
\end{document}
The algorithmicx
bundle have pretty good documentation with a number of examples, I suggest you take a look at it.
Best Answer
Here's an option that provides feasible output. I've removed the physical numbering in the caption:
A newly-defined
\Inputs
and (equivalent)\Initialize
takes one argument that sets its contents in a\Statex
with an indent of\algorithmicindent
- the default - to align with the other algorithm elements. The contents is stored in at
op-aligned\parbox
that fits across 80% of the line. I'm sure you won't need more, but in case you do, that can also be fixed to not cause overfull boxes.