Full implementation of fwcentroid

[alexhelms]

Hi, thank you for creating this library!

I've been using poppy to generate stars for validation of some algorithms. I wanted to implement the fwcentroid function in C# but I noticed it is not fully implemented. I found the IDL code on your STScI site but as mentioned in the python comments, it is identical. I tried searching for the JWST technical reports JWST-STScI-001117 and JWST-STScI-001134 for details but am unable to find those anywhere online.

So that said, my question is are the technical reports and algorithm publicly available? If so, can you point me to them?

Thanks again!

[mperrin]

Hi @alexhelms, I'm not quite sure what you're referring to that's not fully implemented? I'll admit I have not looked at this code in a long time (~a decade) so the details are not fresh in my memory, to say the least.

I can dig up the old technical reports and share if needed. But I don't recall there being any significant part that was left out..

[alexhelms]

I'm referring to comments like

poppy/poppy/fwcentroid.py

Line 78 in a276feb

raise NotImplementedError("Checkbox smoothing not done yet")

and sections of commented out code like

poppy/poppy/fwcentroid.py

Line 164 in a276feb

# XLOC = i - int(XCEN) + XHW + 2

Perhaps these notes are just old and not relevant anymore but I figured I should ask. Thanks again.