Skip to content

Mismatch: PRScsx.py logic does not support multi-population input as described in README #75

@KrysT4L75

Description

@KrysT4L75

Dear Author,

I am writing to report a severe discrepancy between the documentation (README.md) and the actual source code (PRScsx.py) provided in this repository.

The Issue: The README clearly instructs users to input multiple populations (e.g., --pop=EUR,EAS) and multiple summary statistic files. However, the PRScsx.py script in the repository appears to contain single-population logic (likely from the original PRS-CS), which fails to loop through multiple populations or read the corresponding reference panels.

Evidence:

README Instruction: The README explicitly states:

python PRScsx.py ... --pop=EUR,EAS ...

Code Logic (Current Version): Upon inspecting the source code after encountering multiple UnboundLocalError and IndexError crashes, I found that the main() function rigidly looks for a single reference file (e.g., snpinfo_mult_1kg_hm3) and lacks the necessary loop (for pp in range(n_pop)) to handle multiple populations.

Request: Could you please verify if the wrong version of PRScsx.py was accidentally pushed to the master branch? Please update the repository with the correct cross-population compatible script as soon as possible.

Best regards,
kryst4l

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions