search

opengeospatial / geotiff

18 stars 9 forks source link

Issue-52 first commit #65

Closed cmheazel closed 5 years ago

cmheazel commented 5 years ago

This commit should apply all comments and chages from GeoTIFF.2018-12-18.RL.clause.7.CRS.edits_remarks_ERO_RLcomment.docx. Of note: GTCitationGeoKey has been incorporated into CitationGeoKey. Requirement names in UnitsGeoKey have been adjusted to better align with the rest of the units terminology.

EmDevys commented 5 years ago

Chuck I am not sure this commit on the withdrawal of GTCitationGeoKey from Citation GeoKeys in the text of requirement 7 is correct. This was decided by Even and myself, after some discussion with Roger, and the Citation_Keys (requirements_class_CitationGeoKeys.adoc) reflects this! More generally, in all the asciidoc documents, the input docs from Roger on Clause 7, Annexes b to e are to be considered as inputs that have been included, but processed after some additional decisions (adjustments, simplification of structure for allowing not to overflow the title levels allowed by asciidoc, ..). And last but not the least, absolute reference from the Word such as 7.4 should be changed to relative internal asciidoc references (as the structure of the Word document has been changed, and is going at too high levels of titles that are not reasonable).

Hoping there is consensus on this. Otherwise please notify.

Emmanuel

De : Chuck Heazel [mailto:notifications@github.com] Envoyé : vendredi 11 janvier 2019 17:48 À : opengeospatial/geotiff Cc : Subscribed Objet : [opengeospatial/geotiff] Issue-52 first commit (#65)

This commit should apply all comments and chages from GeoTIFF.2018-12-18.RL.clause.7.CRS.edits_remarks_ERO_RLcomment.docx. Of note: GTCitationGeoKey has been incorporated into CitationGeoKey. Requirement names in UnitsGeoKey have been adjusted to better align with the rest of the units terminology.

You can view, comment on, or merge this pull request online at:

https://github.com/opengeospatial/geotiff/pull/65

Commit Summary

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/opengeospatial/geotiff/pull/65, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AIZzavC6CV-SwPddkNooQojKdxVquuzuks5vCMBTgaJpZM4Z7z66.

EmDevys commented 5 years ago

Emmanuel,

This commit represents my best effort at extracting the consensus from the various discussions which have taken place. I’m sure I have some of it wrong. But what better way to find out than to publish it and see who objects?

I’m still confused on GTCitationGeoKey. In this commit I deleted GCCitationGeoKey as a separate section and added it to the CitationGeoKey requirement. It was not included before. When I made this modification I overlooked the supporting text. So it’s not that I deleted GTCitationGeoKey. It was not there in the first place.

So I think what I need to do is add GTCitationGeoKey to the paragraph after the Citation GeoKeys header. Correct? What text should I put here?

Yes, the references need to be fixed. Not my top priority however. Content comes first.

Chuck

From: Emmanuel Devys Emmanuel.Devys@ign.fr Sent: Friday, January 11, 2019 5:18 PM To: opengeospatial/geotiff reply@reply.github.com; opengeospatial/geotiff geotiff@noreply.github.com Cc: Subscribed subscribed@noreply.github.com; Charles Heazel CHeazel@wiscenterprises.com; Even Rouault (even.rouault@spatialys.com) even.rouault@spatialys.com; Roger Lott (EPSG) epsg.rl@btinternet.com Subject: RE: [opengeospatial/geotiff] Issue-52 first commit (#65)

Chuck I am not sure this commit on the withdrawal of GTCitationGeoKey from Citation GeoKeys in the text of requirement 7 is correct. This was decided by Even and myself, after some discussion with Roger, and the Citation_Keys (requirements_class_CitationGeoKeys.adoc) reflects this! More generally, in all the asciidoc documents, the input docs from Roger on Clause 7, Annexes b to e are to be considered as inputs that have been included, but processed after some additional decisions (adjustments, simplification of structure for allowing not to overflow the title levels allowed by asciidoc, ..). And last but not the least, absolute reference from the Word such as 7.4 should be changed to relative internal asciidoc references (as the structure of the Word document has been changed, and is going at too high levels of titles that are not reasonable).

Hoping there is consensus on this. Otherwise please notify.

Emmanuel

De : Chuck Heazel [mailto:notifications@github.com] Envoyé : vendredi 11 janvier 2019 17:48 À : opengeospatial/geotiff Cc : Subscribed Objet : [opengeospatial/geotiff] Issue-52 first commit (#65)

This commit should apply all comments and chages from GeoTIFF.2018-12-18.RL.clause.7.CRS.edits_remarks_ERO_RLcomment.docx. Of note: GTCitationGeoKey has been incorporated into CitationGeoKey. Requirement names in UnitsGeoKey have been adjusted to better align with the rest of the units terminology.

You can view, comment on, or merge this pull request online at:

https://github.com/opengeospatial/geotiff/pull/65https://url.emailprotection.link/?aZyQRg2CGut2qgyHrdHxA3mwuOwIYbqj2znY7JL5Xh4JH_sCutkg60GKn8jq5SkTrA3SQsM3Btx2wXqvNgEY1Bg~~

Commit Summary

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://url.emailprotection.link/?aZyQRg2CGut2qgyHrdHxA3mwuOwIYbqj2znY7JL5Xh4JH_sCutkg60GKn8jq5SkTrA3SQsM3Btx2wXqvNgEY1Bg~~, or mute the threadhttps://url.emailprotection.link/?aZyQRg2CGut2qgyHrdHxA3sf8P0YgATis_lfPk4hOlJPhKk_g1mf5WquDHFVE_sJIrINd6Oes8K78xsJbzJQ_xlA9vBa_odBmJMDE_AEFKY8qZiNTeCKY-E-8mhBwAVivAdYTijATCfB0HBw-_vIpUxs8HHihZb3tdG2VBxsZw9s~.

EmDevys commented 5 years ago

Chuck I understand, and we were probably not clear enough (not enough information / communication from us) on the work done with Even (and a short telecon with Roger) on 3rd Jan. on basis of the Word document, and in the discussions by emails in the following days.

As I should review Clause 7 and Annexes B-E this weekend (and hopefully adjust the references), if you wish, I can re-establish the text by myself on Clause 7 for the Citation GeoKeys.

As you put it, content is the priority, in order to be able to elaborate a 1st exhaustive draft of the 1.1 version).

Emmanuel

De : Charles Heazel [mailto:CHeazel@wiscenterprises.com] Envoyé : samedi 12 janvier 2019 01:06 À : Emmanuel Devys; opengeospatial/geotiff; opengeospatial/geotiff Cc : Subscribed; Even Rouault (even.rouault@spatialys.com); Roger Lott (EPSG) Objet : RE: [opengeospatial/geotiff] Issue-52 first commit (#65)

Emmanuel,

This commit represents my best effort at extracting the consensus from the various discussions which have taken place. I’m sure I have some of it wrong. But what better way to find out than to publish it and see who objects?

I’m still confused on GTCitationGeoKey. In this commit I deleted GCCitationGeoKey as a separate section and added it to the CitationGeoKey requirement. It was not included before. When I made this modification I overlooked the supporting text. So it’s not that I deleted GTCitationGeoKey. It was not there in the first place.

So I think what I need to do is add GTCitationGeoKey to the paragraph after the Citation GeoKeys header. Correct? What text should I put here?

Yes, the references need to be fixed. Not my top priority however. Content comes first.

Chuck

From: Emmanuel Devys Emmanuel.Devys@ign.fr Sent: Friday, January 11, 2019 5:18 PM To: opengeospatial/geotiff reply@reply.github.com; opengeospatial/geotiff geotiff@noreply.github.com Cc: Subscribed subscribed@noreply.github.com; Charles Heazel CHeazel@wiscenterprises.com; Even Rouault (even.rouault@spatialys.com) even.rouault@spatialys.com; Roger Lott (EPSG) epsg.rl@btinternet.com Subject: RE: [opengeospatial/geotiff] Issue-52 first commit (#65)

Chuck I am not sure this commit on the withdrawal of GTCitationGeoKey from Citation GeoKeys in the text of requirement 7 is correct. This was decided by Even and myself, after some discussion with Roger, and the Citation_Keys (requirements_class_CitationGeoKeys.adoc) reflects this! More generally, in all the asciidoc documents, the input docs from Roger on Clause 7, Annexes b to e are to be considered as inputs that have been included, but processed after some additional decisions (adjustments, simplification of structure for allowing not to overflow the title levels allowed by asciidoc, ..). And last but not the least, absolute reference from the Word such as 7.4 should be changed to relative internal asciidoc references (as the structure of the Word document has been changed, and is going at too high levels of titles that are not reasonable).

Hoping there is consensus on this. Otherwise please notify.

Emmanuel

De : Chuck Heazel [mailto:notifications@github.com] Envoyé : vendredi 11 janvier 2019 17:48 À : opengeospatial/geotiff Cc : Subscribed Objet : [opengeospatial/geotiff] Issue-52 first commit (#65)

This commit should apply all comments and chages from GeoTIFF.2018-12-18.RL.clause.7.CRS.edits_remarks_ERO_RLcomment.docx. Of note: GTCitationGeoKey has been incorporated into CitationGeoKey. Requirement names in UnitsGeoKey have been adjusted to better align with the rest of the units terminology.

You can view, comment on, or merge this pull request online at:

https://github.com/opengeospatial/geotiff/pull/65https://url.emailprotection.link/?aZyQRg2CGut2qgyHrdHxA3mwuOwIYbqj2znY7JL5Xh4JH_sCutkg60GKn8jq5SkTrA3SQsM3Btx2wXqvNgEY1Bg~~

Commit Summary

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://url.emailprotection.link/?aZyQRg2CGut2qgyHrdHxA3mwuOwIYbqj2znY7JL5Xh4JH_sCutkg60GKn8jq5SkTrA3SQsM3Btx2wXqvNgEY1Bg~~, or mute the threadhttps://url.emailprotection.link/?aZyQRg2CGut2qgyHrdHxA3sf8P0YgATis_lfPk4hOlJPhKk_g1mf5WquDHFVE_sJIrINd6Oes8K78xsJbzJQ_xlA9vBa_odBmJMDE_AEFKY8qZiNTeCKY-E-8mhBwAVivAdYTijATCfB0HBw-_vIpUxs8HHihZb3tdG2VBxsZw9s~.