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

Format suggestion. #8

Merged
merged 4 commits into from
Jan 30, 2015
Merged

Conversation

ehrenfeu
Copy link
Contributor

I've rearranged the existing three CSV files into something that should be robust and easy to parse as a suggestion for issue #5.

Rationale:

  • Items with different semantics should not be mixed in one file. In other words: don't put the "section names" of the different regions into the same CSV file, as every parser will have to guess if this is a line describing an actual chart sheet, or if it is a section header.
  • Different regions go into differen sub-directories.
  • Every directory has an index file.
  • All other files in a directory have a consistent syntax.

All of these points are particularly targeted towards writing a simple validator that could be used to check the syntax of those files and test them for meaningful content (e.g. valid coordinates, etc.).

Please have a look at this format and let me know what you think!

@ehrenfeu
Copy link
Contributor Author

To see how the CSV files actually look like with the GitHub renderer, just have a look at this branch in my repository:

https://github.com/ehrenfeu/oseam_chart-sheet-properties/tree/separate_sections/IHO_PUB_S-11

ehrenfeu added a commit that referenced this pull request Jan 30, 2015
@ehrenfeu ehrenfeu merged commit 259b143 into OpenSeaMap:master Jan 30, 2015
@ehrenfeu ehrenfeu deleted the separate_sections branch February 6, 2015 22:40
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.

1 participant