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

remove check for forcing csv files #239

Merged
merged 1 commit into from
Nov 26, 2024

Conversation

JoshCu
Copy link
Collaborator

@JoshCu JoshCu commented Nov 26, 2024

fixes #238

@manjilasingh
Copy link

I tested the changes but i am still getting the same error

image

@JoshCu
Copy link
Collaborator Author

JoshCu commented Nov 26, 2024

I tested the changes but i am still getting the same error

image

Did you rebuild the docker image to test it? The hellongen script is inside the docker container so it won't be updated unless the image is rebuilt.

You could test it manually though which would probably be easier. Run the guide script as normal but select 3 to run the bash shell.
You can then modify the script with vim /ngen/HelloNGEN.sh
Then while still on the bash shell, run /ngen/HelloNGEN.sh /ngen/ngen/data to continue running the modified script

@benlee0423
Copy link

@manjilasingh
Build is running now. And I will let you know if it is done, and instructions to run it.
Stay tuned.

@manjilasingh
Copy link

I tested it again with the rebuilt Docker image, and it works now.

@benlee0423 benlee0423 merged commit 4301377 into CIROH-UA:main Nov 26, 2024
7 of 9 checks passed
@benlee0423
Copy link

Thank you @JoshCu for quick fix.

@JoshCu JoshCu deleted the wsl_parallel_fix branch November 27, 2024 00:32
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.

Error when running in parallel mode with AWI_16_2863657_007 input data on Windows
3 participants