calico / basenji

Sequential regulatory activity predictions with deep convolutional neural networks.
Apache License 2.0
395 stars 121 forks source link

failed running "basenji_sat_bed.py" in tutorials #152

Closed yal054 closed 1 year ago

yal054 commented 1 year ago

Hi, thank you for maintaining this great package.

I have error when running the "basenji_sat_bed.py" in tutorials

model_strides [128] target_lengths [1024] target_crops [0] Predicting 0 2023-03-13 13:25:22.633776: I tensorflow/stream_executor/cuda/cuda_dnn.cc:368] Loaded cuDNN version 8202 2023-03-13 13:25:25.229024: I tensorflow/stream_executor/cuda/cuda_blas.cc:1786] TensorFloat-32 will be used for the matrix multiplication. This will only be logged once. ERROR: Sequence 0 failed Waiting for threads to finish.

Is that a communicate error? Could you let me know how to fix it? Thanks!

yal054 commented 1 year ago

I think I figured, the "ScoreWorker" is expecting 4 "seq_pred_stats" items to be passed to it, but "seq_preds_js" is missing in the script.

lihaicheng7003 commented 1 year ago

I encountered the same problem, did you solve the problem?

lihaicheng7003 commented 1 year ago

After removing "seq_pred_stats", it seems to work fine

davek44 commented 1 year ago

You're right. I added a new score to basenji_sat_vcf without updating basenji_sat_bed. I pushed a fix to the master branch. Thanks!