Skip to content
This repository has been archived by the owner on Jan 3, 2022. It is now read-only.

oil-lib: alias oils for api compatability #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gauteh
Copy link
Member

@gauteh gauteh commented Sep 16, 2021

No description provided.

@knutfrode
Copy link

Did you duplicate an oil, so that there are two lines with identical data and ID's, but different names? Does that work in practice?

@gauteh
Copy link
Member Author

gauteh commented Sep 16, 2021 via email

@knutfrode
Copy link

knutfrode commented Sep 16, 2021

Sounds dangerous! Could be the start of the end of the world. As we know it.

@knutfrode
Copy link

Instead of fixing this in ADIOS, I could make a mapping in OpenOil: OldOilNames -> CorrespondingNewOilNames

@gauteh
Copy link
Member Author

gauteh commented Sep 16, 2021

Yes, I am also doing this in the new version: https://github.com/OpenDrift/opendrift/pull/685/files#diff-bd0f1c9292901469fefb1fccf0f9d0b5e8a99e07c6d422218688032ea9c5e8dbR27 -- the main reason for not doing that now is:

  • it will probably be a bigger change and need more tests
  • the adios oil library is ours and is going to be discontinued anyway

otherwise: I agree it is a better solution !

-- gaute

@knutfrode
Copy link

Ok, so the mapping from GENERIC -> "some specific oil" will be in OpenOil, and not in ADIOS itself?

@gauteh
Copy link
Member Author

gauteh commented Sep 16, 2021 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants