Skip to content

Tests fail due to failed dependency with pyplomb #177

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
ArnauMiro opened this issue Apr 11, 2025 · 5 comments
Open

Tests fail due to failed dependency with pyplomb #177

ArnauMiro opened this issue Apr 11, 2025 · 5 comments
Assignees
Labels
bug Something isn't working testsuite Improvements on the testsuite of pyLOM to make it more robust

Comments

@ArnauMiro
Copy link
Owner

@bef-18 take a look at this:

ModuleNotFoundError: No module named 'pyplomb'
@ArnauMiro
Copy link
Owner Author

@bef-18 and since you're at it:

[ 2/12] Cythonizing pyLOM/POD/wrapper.pyx
warning: pyLOM/POD/wrapper.pyx:67:24: local variable 'X_var' referenced before assignment
warning: pyLOM/POD/wrapper.pyx:68:24: local variable 'X_var' referenced before assignment
warning: pyLOM/POD/wrapper.pyx:69:8: local variable 'X_var' referenced before assignment

You're missing to allocate X_var.

@ArnauMiro ArnauMiro added the bug Something isn't working label Apr 14, 2025
@ArnauMiro
Copy link
Owner Author

Oh, and @bef-18 we need a testsuite run for PCA.

@ArnauMiro ArnauMiro added the testsuite Improvements on the testsuite of pyLOM to make it more robust label Apr 14, 2025
@ArnauMiro
Copy link
Owner Author

And also check this compilation warning:

pyLOM/POD/wrapper.c: In function ‘__pyx_pf_5pyLOM_3POD_7wrapper__srun’:
pyLOM/POD/wrapper.c:20862:7: warning: implicit declaration of function ‘snorm_variance’; did you mean ‘stemporal_variance’? [-Wimplicit-function-declaration]
20862 |       snorm_variance(__pyx_v_Y, __pyx_v_Z, __pyx_v_X_var, __pyx_v_m, __pyx_v_n);
      |       ^~~~~~~~~~~~~~
      |       stemporal_variance
pyLOM/POD/wrapper.c: In function ‘__pyx_pf_5pyLOM_3POD_7wrapper_2_drun’:
pyLOM/POD/wrapper.c:21699:7: warning: implicit declaration of function ‘dnorm_variance’; did you mean ‘dtemporal_variance’? [-Wimplicit-function-declaration]
21699 |       dnorm_variance(__pyx_v_Y, __pyx_v_Z, __pyx_v_X_var, __pyx_v_m, __pyx_v_n);
      |       ^~~~~~~~~~~~~~
      |       dtemporal_variance
In function ‘__pyx_pf_5pyLOM_3POD_7wrapper__srun’,
    inlined from ‘__pyx_pw_5pyLOM_3POD_7wrapper_1_srun’ at pyLOM/POD/wrapper.c:20492:13:
pyLOM/POD/wrapper.c:20853:7: warning: ‘__pyx_v_X_var’ may be used uninitialized [-Wmaybe-uninitialized]
20853 |       stemporal_variance(__pyx_v_X_var, (&(*((float *) ( /* dim=1 */ (( /* dim=0 */ (__pyx_v_X.data + __pyx_t_13 * __pyx_v_X.strides[0]) ) + __pyx_t_14 * __pyx_v_X.strides[1]) )))), __pyx_v_X_mean, __pyx_v_m, __pyx_v_n);
      |       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
pyLOM/POD/wrapper.c: In function ‘__pyx_pw_5pyLOM_3POD_7wrapper_1_srun’:
pyLOM/POD/wrapper.c:20512:10: note: ‘__pyx_v_X_var’ was declared here
20512 |   float *__pyx_v_X_var;
      |          ^~~~~~~~~~~~~

@bef-18
Copy link
Collaborator

bef-18 commented Apr 15, 2025

PCA just takes the run from POD and does a vecmat product between S and V to compute the scores. Then, the POD test should be enough to validate the PCA module.

@ArnauMiro
Copy link
Owner Author

Yeah, however, the normalization by the variance is not tested and thus we should create a test that does that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working testsuite Improvements on the testsuite of pyLOM to make it more robust
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

2 participants