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

Discussion: provider flat-file representation #68

Closed

Conversation

thekaveman
Copy link
Collaborator

There has been a lot of interest in this over at #58. Here's one starting point to hopefully further the discussion.

The major changes of note:

As part of addressing the desire to have a well-defined object format, .csv files in the models/ directory describe the fields, types, requirements, etc. for each (rather than tables in the README). This is still a little hand-wavy in my opinion, but at least its machine-readable 🤷‍♂️. Not sure if/how this effort can intersect with #46 and its current proposal #53

@hunterowens hunterowens added this to the 0.2.0 milestone Sep 14, 2018
@ezheidtmann
Copy link

Interesting. Just a half-baked thought here, but what if we support the CSV use-case by providing an open-source transform tool that produces CSV based on JSON input?

I like the idea of machine-readable schema, but perhaps we should use an established schema language like JSON Schema. Unless the CSV schema format you use is a standard already?

@migurski
Copy link

@ezheidtmann A transform tool could temporarily bridge this gap. I’d like to see it officially supported as a client-side tool on an MDS spec page, maybe found inside this repo? It would require that any MDS Provider API endpoint be fully CORS-compliant.

@hunterowens
Copy link
Collaborator

👍 to the idea of a transformation tool rather than a CSV endpoint. @ezheidtmann @migurski.

@ezheidtmann
Copy link

Great -- that tool can be developed when a city needs it.

@thekaveman
Copy link
Collaborator Author

It'll still be here for folks to comment on... but I'm closing this PR as we look towards different approaches.

@thekaveman thekaveman closed this Sep 19, 2018
@thekaveman thekaveman deleted the flat-file branch February 21, 2020 19:20
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.

4 participants