Inconsistency between TrainDataset and EvalDataset's code

[jsalzRR]

Hi, I happened to stumble upon this :
In TrainDataset.get_render :

            # Match camera space to image pixel space
            scale_intrinsic = np.identity(4)
            scale_intrinsic[0, 0] = scale / ortho_ratio
            scale_intrinsic[1, 1] = -scale / ortho_ratio
            scale_intrinsic[2, 2] = scale / ortho_ratio

In EvalDataset.get_render :

            # Match camera space to image pixel space
            scale_intrinsic = np.identity(4)
            scale_intrinsic[0, 0] = scale / ortho_ratio
            scale_intrinsic[1, 1] = -scale / ortho_ratio
            scale_intrinsic[2, 2] = -scale / ortho_ratio

Is the fact that there is an additional minus sign in EvalDataset wanted ? If it is, I would be grateful if you could explain why these implementations are different :)

Good day !