search

zhilizheng / SBayesRC

GNU General Public License v3.0
25 stars 5 forks source link

An extra column 'WD' in the LD data is required for impute() #4

Closed shinichinamba closed 1 year ago

shinichinamba commented 1 year ago

Dear Zhili,

I found that impute() caused an error because it internally tried to rename the column name of $LD_PATH/snp.info, but the length of the new column names (9) did not match the actual length (8). This was because the new column names included an extra name, 'WD'; it seemed that this column was not used in any functions. I observed this issue when using UKB EAS imputed LD data.

(Although it was not a straightforward solution, I've added a dummy column to $LD_PATH/snp.info and successfully imputed the summary statistics.)

Best,

Shinichi

zhilizheng commented 1 year ago

Hi Shinichi,

thanks for the report and sorry for the late reply. I have find the issue in our new LD files in EAS (no genetic distance).

I will fix the code soon. for your running, you can add extra column into the $LD_PATH/snp.info file before the BP column.

Regards, Zhili

zhilizheng commented 1 year ago

Hi Shinichhi,

I have fixed the issue in v0.1.4b. Thanks for the report. Let me know if it has further issues.

Regards, Zhili