search

diku-dk / bfast

GPU Implementation for BFAST
GNU General Public License v3.0
37 stars 17 forks source link

IndexError: index -54 is out of bounds #7

Closed Carolina710 closed 3 years ago

Carolina710 commented 3 years ago

Hello,

I share "Tile_1" data: https://drive.google.com/drive/folders/15DK5T9hZUaaIBNsfzR3tTp5pFb9XCWt5?usp=sharing

Using this data, an error occurs while processing row 120, in the Python version. This error does not always happen, in this case, it happens when I use start_monitor = datetime(2018, 6, 18), for example. If you try start_monitor = datetime(2016, 1, 1) everything work fine.

The error is the follow:

image

But running in OpenCL I can obtain results.

You can use this gist: https://gist.github.com/Carolina710/330b473478dd59188edaaa6a3fd76bb6 to work with Tile_1.

Thank you!

mortvest commented 3 years ago

I have run the gist that you have provided. The error does not seem to occur on the develop branch. It was probably fixed with the other bugs with the python backend. Could you check if it is the case for you as well?

Carolina710 commented 3 years ago

Hi! Yes, it works well for me too. Thank you very much, and thank you for adding the magnitude calculation! I will continue to work around the library, and any other issues I may find, I will let you know!

mortvest commented 3 years ago

fixed