Giter Site home page Giter Site logo

Comments (6)

francois-a avatar francois-a commented on July 22, 2024 1

Great, thanks for confirming. I recommend filtering those out for trans-QTL mapping.

from tensorqtl.

francois-a avatar francois-a commented on July 22, 2024

Hi,
Thanks for pointing this out. This has been on the to-do list for a while. Added in 1ba37bb; the output (when return_sparse = True) now includes b and b_se.

from tensorqtl.

JonMarten avatar JonMarten commented on July 22, 2024

Hi Francois, thanks for adding this.

I've noticed that as well as betas, the p-values have changed.

Using the same script each time, tensorqtl obtained through pip install tensorqtl, reports 8 SNPs as significant, but the version from pip install git+https://github.com/broadinstitute/tensorqtl.git#egg=tensorqtl gives no SNPs as significant. All other parameters and input data are identical. (It also runs a lot faster!)

Has the calculation of p-values changed?

from tensorqtl.

francois-a avatar francois-a commented on July 22, 2024

Hi,
Can you please provide specific data to reproduce the issues you're reporting?

The p-value calculation has not changed, and the output should be identical between versions. It's also unclear why you're seeing speed differences. I just re-tested these versions using the GEUVADIS example and am getting close to identical runtimes.

The only change in the latest commit is the addition of an imputation step for missing values at 1ba37bb#diff-c20eed07a12120ef8f33386bed4c6603R101. I added this for consistency with the cis-QTL code. This would change the p-values for variants with missing values, but such variants should generally be excluded from trans-QTL mapping (before running tensorQTL) due to the higher risk of being artifacts.

from tensorqtl.

JonMarten avatar JonMarten commented on July 22, 2024

Thanks - I wonder if that could be it, I'll check. I have the usual data protection problems but if it's not being driven by missingness I'll try and recreate with toy or randomised data and share here.

As for the speed, if it's nothing you've changed, I strongly suspect it's just our cluster's quirky file system.

from tensorqtl.

JonMarten avatar JonMarten commented on July 22, 2024

So I've checked by manually setting missing SNPs to different values and this imputation is definitely what's driving the effect I'm observing. Thanks!

from tensorqtl.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.