-
Notifications
You must be signed in to change notification settings - Fork 61
Open
Labels
bug 🐛Something isn't workingSomething isn't working
Description
The linked code drops the non-logarithmic column and renames the logarithmic ones. If someone is testing different distributions (logarithmic and non-logarithmic), the non-logarithmic distributions will be lost. I suggest that we short term skip dropping the non-logarithmic even though it creates some duplicates. Could be better to e.g. only keep the non-logarithmic which exists independent of chosen distribution, but instead have an option in the gui to use a logarithmic axis.
webviz-subsurface/webviz_subsurface/_models/parameter_model.py
Lines 112 to 119 in 1b1bb17
| # Keep only LOG parameters | |
| log_params = [param for param in self._dataframe if param.startswith("LOG10_")] | |
| self._dataframe = self._dataframe.drop( | |
| columns=[param.replace("LOG10_", "") for param in log_params] | |
| ) | |
| self._dataframe = self._dataframe.rename( | |
| columns={col: f"{col} (log)" for col in log_params} | |
| ) |
Metadata
Metadata
Assignees
Labels
bug 🐛Something isn't workingSomething isn't working
Type
Projects
Status
Backlog 📝