looper isn't working with sample_table_index

[nsheff]

Using a basic project with a sample table index fails, due to this line:

looper/looper/looper.py

Line 458 in 1468956

name=sample.sample_name,

it's expecting there to be a sample_name column, which there is not, since I'm using sample_table_index.

This makes looper not fully compliant with PEP.

[donaldcampbelljr]

I believe this is a Peppy issue: pepkit/peppy#459

[nsheff]

I don't think it's a peppy issue -- it's looper code that is referring to sample.sample_name

are you suggesting that peppy make available a .sample_name attribute to refer to wherever the index is?

[donaldcampbelljr]

I don't think it's a peppy issue -- it's looper code that is referring to sample.sample_name

are you suggesting that peppy make available a .sample_name attribute to refer to wherever the index is?

Yes. I thought that was the original functionality.

Some digging, for reference to this issue:
pepkit/peppy@3d5495b

[donaldcampbelljr]

I added a commit here where I made a test to help troubleshoot this issue further: 8fde43a

Using a sample_table_index in the project config, I also replaced sample.sample_name in the Looper code with sample[self.prj.sample_table_index]. This was done based on previous, similar work that may not have been completed: 9ce510d

However, I now see an Eido validation error during the test where I simply attempt to use looper run

Error: EidoValidationError (Validation failed): {"'sample_name' is a required property": [{'type': "'sample_name' is a required property", 'message': "'sample_name' is a required property on instance project", 'sample_name': 'project'}]}
["'sample_name' is a required property"]

Related test data:

project config:

name: looper_advanced_test
pep_version: 2.0.0
sample_table: annotation_sheet.csv
sample_modifiers:
  append:
    attr: val
  derive:
    attributes:
    - read1
    - read2
    sources:
      SRA_1: '{SRR}_1.fastq.gz'
      SRA_2: '{SRR}_2.fastq.gz'
sample_table_index: NOT_SAMPLE_NAME

annotation sheet

NOT_SAMPLE_NAME	protocol	data_source	SRR	Sample_geo_accession	read1	read2
sample1	PROTO1	SRA	SRR5210416	GSM2471255	SRA_1	SRA_2
sample2	PROTO1	SRA	SRR5210450	GSM2471300	SRA_1	SRA_2
sample3	PROTO2	SRA	SRR5210398	GSM2471249	SRA_1	SRA_2

[donaldcampbelljr]

Future discussion to happen in: pepkit/peppy#459

However, will keep this open to track for Looper.

[donaldcampbelljr]

Ok, I believe this PR in Peppy (pepkit/peppy#484) will solve this issue and the corresponding issue: pepkit/peppy#459

• We will need to do a release of Peppy (or a pre-release) and pin that in Looper reqs.