Skip to content
This repository has been archived by the owner on Aug 29, 2023. It is now read-only.

Merge meta_data and meta_info #301

Closed
forman opened this issue Jul 20, 2017 · 1 comment
Closed

Merge meta_data and meta_info #301

forman opened this issue Jul 20, 2017 · 1 comment
Assignees
Labels
Milestone

Comments

@forman
Copy link
Member

forman commented Jul 20, 2017

Expected behavior

There should be a single property in a (local?) data source's JSON representation that provides the data source's meta information, e.g. meta_info as defined in the DataSource interface.

Actual behavior

We currently find the JSON properties meta_data and meta_info. Why keep two?

See code

Specifications

Cate 0.9.0.dev4

@forman forman added this to the IPM6 milestone Jul 20, 2017
kbernat pushed a commit that referenced this issue Sep 18, 2017
kbernat pushed a commit that referenced this issue Sep 19, 2017
kbernat pushed a commit that referenced this issue Sep 19, 2017
kbernat pushed a commit that referenced this issue Sep 19, 2017
kbernat pushed a commit that referenced this issue Sep 20, 2017
kbernat pushed a commit that referenced this issue Sep 20, 2017
kbernat pushed a commit that referenced this issue Sep 20, 2017
kbernat pushed a commit that referenced this issue Sep 20, 2017
kbernat pushed a commit that referenced this issue Sep 20, 2017
kbernat pushed a commit that referenced this issue Sep 20, 2017
kbernat pushed a commit that referenced this issue Sep 20, 2017
kbernat pushed a commit that referenced this issue Sep 20, 2017
@kbernat kbernat closed this as completed Sep 20, 2017
@kbernat
Copy link
Collaborator

kbernat commented Sep 20, 2017

Still possible to read obsolete format, new datasets use only new/common meta_info structure.

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

No branches or pull requests

2 participants