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

Feature: Make xtgeo dependency optional #335

Closed
oysteinequinor opened this issue Jul 2, 2024 · 2 comments
Closed

Feature: Make xtgeo dependency optional #335

oysteinequinor opened this issue Jul 2, 2024 · 2 comments
Assignees

Comments

@oysteinequinor
Copy link

xtgeo is a rather large and complicated dependency. We are having big issues with building on Mac M-series and issues due to xtgeo pinning certain versions of dependencies.

We are not using any xtgeo functionality in fmu_sumo, and would like to avoid dealing with it.

Would you be open to making the xtgeo dependency optional?

@equinor-ruaj
Copy link
Contributor

Looking into:

  • Make xtgeo an optional dependency
  • Lazy loading xtgeo
  • Warning trying to use xtgeo method, but it is not installed
  • Check with xtgeo team: Can they fix the version pinnings that cause trouble?

@oysteinequinor
Copy link
Author

We are using a fork with this commit. Might be a starting point if you want to go in that direction.
oysteinequinor@33eca4d

@rwiker rwiker self-assigned this Sep 12, 2024
@rwiker rwiker mentioned this issue Sep 12, 2024
6 tasks
@equinor-ruaj equinor-ruaj linked a pull request Sep 16, 2024 that will close this issue
6 tasks
@equinor-ruaj equinor-ruaj removed a link to a pull request Sep 16, 2024
6 tasks
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

No branches or pull requests

3 participants