frank ties.method='sequence'?

Open
#4,241 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
1-2 days
Newbie friendliness
42/100
Issue type
Refactor
Clarity
Needs clarification
Activity status
Quiet
Tech stack
r
Domain
data

Research direction

Start by locating the frank signature and searching src/ for SEQUENCE and its callers. Determine whether the unreachable ties.method='sequence' path should be exposed at R level or removed; done means resolving that choice consistently with the existing frank interface.

Written by the indexing model from the issue text.

Description

feature request

I see the following in the code:

else if (!strcmp(pties, "sequence")) ties = SEQUENCE;
// ...
    case SEQUENCE :
      for (int i=0; i<length(xstartArg); ++i) {
        int k=1;
        for (int j=xstart[i]-1; j<xstart[i]+xlen[i]-1; ++j)
          ians[xorder[j]-1] = k++;
      }
      break;

but it's not in the frank signature so there's no way to get there from R level (and it's not called elsewhere in src/).

what's this argument? do we want to support it? or maybe just remove it?

Dominant language
R
Stars
3.9k
Forks
1.1k
Avg merge
14h 4m
Merged PRs (30d)
4

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from Rdatatable/data.table

All issues in Rdatatable/data.table

Similar issues

More R issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.