Skip to content
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

Sparse MFK (SMFK) #717

Closed
wants to merge 8 commits into from
Closed

Sparse MFK (SMFK) #717

wants to merge 8 commits into from

Conversation

mcastanoUQ
Copy link
Contributor

-Addition of predict values and predict variances to MFCK.
-Modification of tutorial MFCK to explain the incorporation of the new functions.
-Creation of new Sparse MFK class using sgp for the lowest fidelity levels.
-Creation of tutorial notebook and test script.

@mcastanoUQ
Copy link
Contributor Author

Hi, some idea of the error?, locally works well
image

@relf
Copy link
Member

relf commented Mar 5, 2025

I've just tested your PR locally and it fails like in CI tests. Are you sure to have committed and pushed the code you've tested ?

@mcastanoUQ
Copy link
Contributor Author

Yes Remy, I just have one folder for the local version, there's no other possible test_smfk.py file. Maybe can be the version of pytest? As in the image I'm using the version 7.4.0

@relf
Copy link
Member

relf commented Mar 6, 2025

What version of numpy and scipy are you using?

@mcastanoUQ
Copy link
Contributor Author

Hi Remmy, my versions of numpy and scipy after running the test_smfk script:
image

@relf relf mentioned this pull request Mar 11, 2025
@relf
Copy link
Member

relf commented Mar 11, 2025

superseded by #718

@relf relf closed this Mar 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants