-
Notifications
You must be signed in to change notification settings - Fork 7
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
Change provenance file to JSON format #184
Comments
I’m not sure it is a good idea to implement this feature right now. Changing the format of the provenance file has a great deal of downstream consequences, and I’m not sure it is a good idea so close to release of our version 1.0.
… On Jun 21, 2021, at 12:43 PM, Don Brower ***@***.***> wrote:
The carrel provenance file is formatted as a tab-seperated-values file. This causes problems with adding or changing metadata fields. It also causes problems with carrels whose search queries contain newlines.
There are many alternate formats we could use. I suggest JSON since it is ubiquitous and is intended as a structured way to send data from one program to another. Most languages have a library to read json files, and there is a command line tool (jq) to pull data out of a JSON formatted file.
This ticket is to change the file to have JSON format.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
I made this ticket to track the problem since I noticed some errors related to it in web gui's carrel scraper. The ticket is not saying when it (or whether) the issue should be addressed. Your concerns are noted, and this should wait until after the release. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The carrel provenance file is formatted as a tab-seperated-values file. This causes problems with adding or changing metadata fields. It also causes problems with carrels whose search queries contain newlines.
There are many alternate formats we could use. I suggest JSON since it is ubiquitous and is intended as a structured way to send data from one program to another. Most languages have a library to read json files, and there is a command line tool (jq) to pull data out of a JSON formatted file.
This ticket is to change the file to have JSON format.
The text was updated successfully, but these errors were encountered: