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

Pin pyarrow install to avoid ngen integration issues #40

Closed
wants to merge 3 commits into from

Conversation

hellkite500
Copy link
Collaborator

Issues in pyarrow and arrows upstream prevent us from using v12.0.0 or greater at this point. This PR will ensure a compatible version is used in the dockerfile.

@arpita0911patel
Copy link
Member

The build is successful with these modifications. However, during testing with the guide.sh script and AWI_03W_113060_001.tar.gz, an error persists, indicating "At declaration of smc_profile size, soil_reservoir.n_soil_layers = 0
Using Routing
./HelloNGEN.sh: line 76: 13 Segmentation fault /dmod/bin/ngen-serial $n1 all $n2 all $n3"

@benlee0423
Copy link

That's the same error I got from past commits in ngen repo.

@arpita0911patel
Copy link
Member

Based on the discussion today morning with Nels, Trupesh, Ben, and James we agreed to merge the PR#40,45,47.

@benlee0423
Copy link

Only PR #45 and #47 are merged on #58.
This merge is not necessary after discussion with everyone. Closing this PR.

@benlee0423 benlee0423 closed this Jan 5, 2024
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.

3 participants