Tabstop not working correctly with regular function

[medwatt]

I have a function which generates a row of nodes for use in LaTex.

local function generate_row_of_nodes(index, columns)
    local nodes = {}
    for j = 1, columns-1 do
        table.insert(nodes, i(j, tostring(j)))
        table.insert(nodes, t(" & "))
    end
    table.insert(nodes, i(columns, tostring(columns)))
    table.insert(nodes, t(" \\\\"))
    return sn(index, nodes)
end

I have another function, which uses the previous function, that generates a table of nodes.

local function generate_table_of_nodes(rows, columns, indent_string)
    local nodes = {}
    for j = 1, rows-1 do
        table.insert(nodes, generate_row_of_nodes(j, columns))
        table.insert(nodes, t({"", ""}))
    end
    table.insert(nodes, generate_row_of_nodes(rows, columns))
    return isn(nil, nodes, "$PARENT_INDENT\t" .. indent_string)
end

Using these two functions, I create a table environment snippet as shown below. I notice that when using tab to jump, I'm not able to jump into all the nodes having the last index generated by the functions. For example, in the case of the generate_table_of_nodes, let's say I generate a 2x2 table of nodes. I am only able to jump to cells 1,1 and 1,2 (row, col). Why is this the case and how can I fix this?

    s(
        { trig = "(%d+)x(%d)table", dscr = "table", regTrig = true },
        fmta(
            [[
                \begin{table}[h]
                    \centering
                    \begin{tabular}{<>}
                        \hline
                        <>
                        \hline
                        <>
                        \hline
                    \end{tabular}
                    \caption{<>}
                    \label{tbl:<>}
                \end{table}
            ]],
            {
                f(function(_, snip)
                    return string.rep("|c", tonumber(snip.captures[2]) - 1) .. "|c|"
                end),

                d(1, function(_, snip)
                    return generate_row_of_nodes(nil, snip.captures[2])
                end),

                d(2, function(_, snip)
                    return generate_table_of_nodes(snip.captures[1], snip.captures[2], "\t")
                end),

                i(3, "caption"),
                i(4, "label"),
            }
        )
    )

[L3MON4D3]

Pretty sure its just that one of these indices is wrong.. Have you tried adding some print to track it down?

[medwatt]

This is what I get when I expand 3x3table. The able is printed correctly, but I cannot jump into column 3 and row 3 of the table.

\begin{table}[h]
    \centering
    \begin{tabular}{|c|c|c|}
        \hline
        1 & 2 & 3 \\
        \hline
        1 & 2 & 3 \\
        1 & 2 & 3 \\
        1 & 2 & 3 \\
        \hline
    \end{tabular}
    \caption{caption}
    \label{tbl:label}
\end{table}

[L3MON4D3]

Ahhhhh it might be that columns is a string

[L3MON4D3]

rows as well

[L3MON4D3]

So tonumber on those should do it (AFAIK lua does this automatically, sometimes, but we don't check that the index is actually a number)

[medwatt]

@L3MON4D3, one last question. The snippet below is the same as the one I provided above except that there's a choice that surrounds the tabular environment with a table environment. I would like the dynamic nodes to be the same for both choices. I know that there are restore nodes, but I don't know exactly how to use them. What would I need to change in the functions generate_row_of_nodes and generate_table_of_nodes so that the two choices have the same nodes?

s(
    { trig = "(%d+)x(%d)table", dscr = "table", regTrig = true },
    {
        c(1,
            {
                sn(1,
                    fmta(
                        [[
                        \begin{table}[h]
                            \centering
                            \begin{tabular}{<>}
                                \hline
                                <>
                                \hline
                                <>
                                \hline
                            \end{tabular}
                            \caption{<>}
                            \label{tbl:<>}
                        \end{table}
                        ]],
                        {
                            f(function(_, snip)
                                return string.rep("|c", tonumber(snip.parent.captures[2]) - 1) .. "|c|"
                            end),

                            d(1, function(_, snip)
                                return generate_row_of_nodes(1, snip.parent.captures[2])
                            end),

                            d(2, function(_, snip)
                                return generate_table_of_nodes(snip.parent.captures[1], snip.parent.captures[2], "\t")
                            end),

                            i(3, "caption"),
                            i(4, "label"),
                        }
                    )
                ), -- end first choice

                sn(1,
                    fmta(
                        [[
                        \begin{tabular}{<>}
                            \hline
                            <>
                            \hline
                            <>
                            \hline
                        \end{tabular}
                        ]],
                        {
                            f(function(_, snip)
                                return string.rep("|c", tonumber(snip.parent.captures[2]) - 1) .. "|c|"
                            end),

                            d(1, function(_, snip)
                                return generate_row_of_nodes(1, snip.parent.captures[2])
                            end),

                            d(2, function(_, snip)
                                return generate_table_of_nodes(snip.parent.captures[1], snip.parent.captures[2])
                            end),
                        }
                    )
                ), -- end second choice

            }
        )

    }
)

[L3MON4D3]

I think you might get away with just wrapping the d(1,..., d(2, ... in restoreNodes, if I'm understanding you correctly

[medwatt]

@L3MON4D3, I can't seem to get it to work. Can you help?

This is what I tried:

{
    f(function(_, snip)
        return string.rep("|c", tonumber(snip.parent.captures[2]) - 1) .. "|c|"
    end),

    r(1, d(1, function(_, snip)
        return generate_row_of_nodes(1, snip.parent.captures[2])
    end)),

    r(2, d(1, function(_, snip)
        return generate_table_of_nodes(snip.parent.captures[1], snip.parent.captures[2], "\t")
    end),

    i(3, "caption"),
    i(4, "label"),
}

[L3MON4D3]

Oh, you're missing the key, try r(i, "node_i", d(1, ...)) with i=1,2