autoRIFT Correct computation of v

Correct computation of v_error

Open mliukis opened this issue 2 years ago • 4 comments

As @alex-s-gardner pointed out, the paper and production code need to be corrected in computations of v_error:

https://github.com/nasa-jpl/autoRIFT/blob/e683e485b0e194a0b63def90ccbdee11ad56fb2a/netcdf_output.py#L697

It should be changed to: v_err = (vx_err.*abs(vx) + vy_err.*abs(vy))./v

Oct 08 '22 22:10 mliukis

I discussed this with Yang and the error formulation in the paper is correct. We may need to update our Mosaic code

Oct 10 '22 14:10 alex-s-gardner

@alex-s-gardner so, the netcdf_ouput.py doesn't need to be updated, correct?

Oct 10 '22 21:10 jhkennedy

@jhkennedy that is correct

Oct 10 '22 21:10 alex-s-gardner

@mliukis when you have a chance can you confirm error calculation in mosaic code

Oct 12 '22 16:10 alex-s-gardner

no action needed

Nov 29 '22 01:11 alex-s-gardner