Skagamaster / Python_Pico_Reader

0 stars 1 forks source link

EPD Ring Matrix #9

Open tlprotzman opened 3 years ago

tlprotzman commented 3 years ago

It appears that only the first 16 elements of the ring sum matrix are being filled by EPD_Hits.generate_epd_hit_matrix. Is it intended that they will be split by east/west (i.e. east hits in the first 16 slots, west hits in the second 16)?

Skagamaster commented 3 years ago

You're right; there are only rings 1-16. EW must be specified. I'll check it out.

Skagamaster commented 3 years ago

OK, this should be fixed. It still needs to be tested; I'll put it in a pull request until it's tested.