Webviz expects different name and tagname

[equinor-ruaj]

Currently we set:
name based on filepath
tagname to "grid" or some "prefix"-"object name"

Webviz expects these to be:
name = parameter name
tagname = grid name
I.e. What today is name should be tagname and tagname should be name

[equinor-ruaj]

This is apparently not an issue for Webviz when equinor/fmu-dataio#803 is solved.

Should we also change name <-> tagname in Sim2sumo @perolavsvendsen?
Or should they be something different entirely to match other fmu data?

[perolavsvendsen]

This relates to equinor/fmu-dataio#794 I think. Basically, there needs to be some sort of coherence between the different parts here. Currently, I think there has been a bit of siloed thinking on different components. This is particularly important for super-central metadata fields such as data.name and data.tagname which everyone is depending on for (almost) uniqueness.

Just to be clear, this is regarding 3D grid data - not timeseries/simulation output, yes?

I think it could makes some sense to use the parameter (name) as data.name, so that could be a good start. And it is probably OK to use the gridname for data.tagname. However this does not scale beyond 1 export of a specific grid, which I think we are well past already. (But ironically, due to lack of master data concepts, we probably call them different things during the same workflow, hence this has not hit.) (Yet.)

But OK, in lack of knowledge or better options right now, let's go with the suggestion from Webviz and see where that takes us.

[equinor-ruaj]

Quickfix here

[perolavsvendsen]

Is this something we could/should test in dev before it hits prod? Is that possible? Not fully sure about the consequences.

[equinor-ruaj]

In the (not so distant) future we probably want to use fmu-dataio? #103

[equinor-ruaj]

Is this something we could/should test in dev before it hits prod? Is that possible? Not fully sure about the consequences.

Which environment is targeted is up to users...
The change will only be available in Komodo bleeding, so stable users won't see any changes.

[perolavsvendsen]

OK I'm thinking perhaps test this a bit in a branch or something before it hits Komodo

[equinor-ruaj]

Tested in dev (see comment in PR)