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

Fix integratedTests fails. #2864

Merged
merged 1 commit into from
Dec 1, 2023
Merged

Conversation

CusiniM
Copy link
Collaborator

@CusiniM CusiniM commented Dec 1, 2023

integratedTests PR: https://github.com/GEOS-DEV/integratedTests/pull/63

@MelReyCG, @klevzoff are we sure that we do not want to accept "x" where x is an integer as a valid format for a string? The error I was getting for the plot level seems to suggest that that's not a valid string.

@CusiniM CusiniM added the ci: run CUDA builds Allows to triggers (costly) CUDA jobs label Dec 1, 2023
@CusiniM CusiniM requested review from rrsettgast and TotoGaz December 1, 2023 21:38
@CusiniM CusiniM self-assigned this Dec 1, 2023
@CusiniM CusiniM added the flag: requires rebaseline Requires rebaseline branch in integratedTests label Dec 1, 2023
@CusiniM CusiniM merged commit 9ff1e41 into develop Dec 1, 2023
@CusiniM CusiniM deleted the cusini1/fixIntegratedTestsFails branch December 1, 2023 21:57
ouassimkh pushed a commit that referenced this pull request Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci: run CUDA builds Allows to triggers (costly) CUDA jobs flag: requires rebaseline Requires rebaseline branch in integratedTests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants