-
Notifications
You must be signed in to change notification settings - Fork 15
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
REV: Set depth as default vertical_domain #807
Conversation
@@ -221,7 +221,7 @@ class ExportData: | |||
Note also: If missing or empty, export() may still be done, but without a | |||
metadata file (this feature may change in future releases). | |||
|
|||
content: Optional, default is "depth". Is a string or a dictionary with one key. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sneaked in a removal of incorrect documentation
1ed1683
to
05933ca
Compare
Associated discussion: https://equinor.slack.com/archives/C06E3UTGTEV/p1727263798090899 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, assuming tests will pass, thanks for fast response 🚀
Force merging this as the only failing test is due to a change in sumo uploader that broke create case metadata |
No description provided.