Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow physical metadata to be defined in a template #107

Open
clnsmth opened this issue Apr 11, 2022 · 5 comments
Open

Allow physical metadata to be defined in a template #107

clnsmth opened this issue Apr 11, 2022 · 5 comments

Comments

@clnsmth
Copy link
Contributor

clnsmth commented Apr 11, 2022

make_eml() requires access to data files so physical metadata (e.g. file size, checksum, etc.) can be calculated and the user doesn't have to deal with this minutiae. In some cases this requirement is unnecessarily prohibitive (e.g. large unchanged data files stored in a remote location) or prevents the user from overriding EAL assumptions (e.g. formatName estimated from file extension).

A possible solution is to store this physical metadata in a new metadata template that can be created before running make_eml() or left absent in the case where a user wants the values calculated for them, as is currently done.

@joefutrelle
Copy link

joefutrelle commented Apr 11, 2022

This solution seems workable to me as we could produce and maintain the template in an automated fashion and then simply drop it in place in our assembly line workflow. It would also allow us to track the template in our GitHub repositories for each package.

@clnsmth
Copy link
Contributor Author

clnsmth commented May 6, 2022

@joefutrelle
Copy link

joefutrelle commented May 6, 2022

Looks good to me. In the scenario where we transfer files to EDI out of band (because we can't host them), would the other entity URL be left blank? Or does EML require it?

@clnsmth
Copy link
Contributor Author

clnsmth commented May 6, 2022

You're right @joefutrelle, in that case the URL field would be blank or NA so the EDI Repository wouldn't try downloading the associated data object.

@joefutrelle
Copy link

Perfect.

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

No branches or pull requests

2 participants