Much-Assembly-Required icon indicating copy to clipboard operation
Much-Assembly-Required copied to clipboard

Radioactive Detection Implemented

Open khalludi opened this issue 5 years ago • 2 comments

I implemented the radioactive detection with alpha particles. I've tested the helper methods separately to make sure they work and the project builds, but I haven't been able to test the handleInterrupt() function in-game. Part of this is due to me still figuring out how to make the cubot move XD.

I know the initial issue stated that there should be beta and gamma radiation, but I'm not sure how all three types of radiation will tie into the game, so I only implemented Alpha Radiation. When/If there is a need for beta and gamma, the code can be written quite easily.

Again, this is a draft PR. If someone can test it in-game before I can and let me know what the correct HW_ID should be, this PR can go into prod.

Let me know your thoughts.

Original Issue #136 .

khalludi avatar Sep 17 '20 02:09 khalludi

Hi, thanks a lot for taking the time to contribute.

Alpha/gamma/beta radiation can be implemented with different 'getCounts()' function (e.g. getAlphaCounts(), getBetaCounts() etc), they also have a different constants for impassable blocks in the path.

I've been meaning to implement unit testing for this kind of changes for a while, I'll build some fixture classes that you will be able to use to test the new hardware module.

In the meantime I noted some little changes you can work on

simon987 avatar Sep 17 '20 12:09 simon987

Implemented suggestions and added Beta and Gamma particle functions.

khalludi avatar Sep 17 '20 14:09 khalludi