Skip to content

add metadata(), to avoid loading the whole file#178

Merged
bjarthur merged 1 commit intoJuliaIO:masterfrom
bjarthur:bja/metadata
May 14, 2018
Merged

add metadata(), to avoid loading the whole file#178
bjarthur merged 1 commit intoJuliaIO:masterfrom
bjarthur:bja/metadata

Conversation

@bjarthur
Copy link
Copy Markdown
Member

@bjarthur
Copy link
Copy Markdown
Member Author

@timholy could you please review this and the two linked PRs? thanks.

@bjarthur bjarthur merged commit aa8f1a6 into JuliaIO:master May 14, 2018
@bjarthur bjarthur deleted the bja/metadata branch May 14, 2018 18:48
@bkamins
Copy link
Copy Markdown

bkamins commented May 26, 2022

@bjarthur, @timholy - I could not find a documentation of contract for metadata function defined in FileIO.jl.

In parallel we are considering adding metadata function to DataAPI.jl.

I have the following questions:

  • would it make sense for FileIO.metadata to add a method to DataAPI.metadata instead of creating a new function;
  • in this case would it make sense/be possible to return e.g. NamedTuple instead of Tuple (in DataAPI.jl a natural interface is to return an object having key => value pairs)

Or maybe there is some other solution to the name clash?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

query metadata feature?

2 participants