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

How should we process the 22 outstanding CoverageJSON issues #11

Closed
chris-little opened this issue Oct 27, 2021 · 13 comments
Closed

How should we process the 22 outstanding CoverageJSON issues #11

chris-little opened this issue Oct 27, 2021 · 13 comments
Assignees
Labels
help wanted Extra attention is needed

Comments

@chris-little
Copy link
Contributor

The original specification has about 20 issues outstanding. We need to discuss which can be addressed and incorporated into the proposed OGC Community Standard (assuming strong backward compatibility) and which are future work or enhancements.

Should we copy them across to here, or leave them in the original repo?

@lewismc
Copy link
Member

lewismc commented Oct 28, 2021

@chris-little +1 for copy here. I'll do that in 72 hrs if there are no objections.
Some basic metadata tagging will allow us to organize and prioritize.

@chris-little
Copy link
Contributor Author

@lewismc +1 to your idea. Tagging or naming them so we can filter them from those required to get the Community Standard across the finishing line.

@chris-little
Copy link
Contributor Author

@lewismc As we are:

  1. Making good progress with addressing the Vote comments;
  2. Producing a schema for conformance testing;
  3. Setting up project framework in the repo;
    could you transfer across the 20 outstanding issues from the original COvJSON repo please?

@chris-little chris-little added the help wanted Extra attention is needed label Feb 16, 2022
@jonblower
Copy link
Contributor

@chris-little @lewismc could we chat about this before transferring the issues please? There may be some that could be closed, or left where they are. I'm just trying to figure out how to best organise things between the original and OGC repos in general. (Sorry I missed the call yesterday.)

@chris-little
Copy link
Contributor Author

chris-little commented Feb 17, 2022

@jonblower @lewismc Does this list of Issue titles help? We could edit while we discuss, then bring across the agreed list.
[Updated with annotations from @letmaik in later comment]
covjson/specification#95 needs discussion
covjson/specification#94 needs discussion
covjson/specification#93 needs discussion
covjson/specification#91 CLOSED
covjson/specification#88 crs 5 tasks
covjson/specification#86 needs discussion
covjson/specification#82 CLOSED
covjson/specification#81 needs discussion
covjson/specification#78 CLOSED
covjson/specification#75 needs discussion CLOSED
covjson/specification#71 needs discussion CLOSED
covjson/specification#66 needs discussion 1.0 CLOSED
covjson/specification#63 needs discussion CLOSED
covjson/specification#61 needs discussion CLOSED
covjson/specification#60
covjson/specification#55 needs discussion 1.0 CLOSED
covjson/specification#45 needs discussion 1.0
covjson/specification#38 needs discussion future CLOSED
covjson/specification#32 needs discussion future CLOSED
covjson/specification#29 needs discussion future CLOSED
covjson/specification#15 needs discussion 1.0 CLOSED
covjson/specification#3 needs discussion future CLOSED

@lewismc
Copy link
Member

lewismc commented Feb 18, 2022

OK to state the obvious it looks like there is a lot of discussion needed. Can the ones without a qualifying comment can be moved over?

@letmaik
Copy link
Contributor

letmaik commented Feb 18, 2022

I commented on and closed 13 of those issues for various reasons: out of scope, not an issue, etc.

The remaining ones are:

Ideally looked at before first release:

Can be deferred to a future version:

@jonblower
Copy link
Contributor

Thanks for doing this @letmaik!

How about we move the "comparison" discussions to this (OGC) repo, as they are not so much about CovJSON itself, but more about its place in the world?

From my point of view, I'd like to discuss 88, 93 and 94 a bit further, as we've had specific user requests about these topics. It would be good to establish whether we need them for a first release.

I agree that the other tickets can wait until a future version. The "multiple time axes" one (number 45) is something that @chris-little has flagged before, so he might want to consider making this in scope for the first release too. But my feeling is that there won't be many people wanting to create CovJSON objects that have two time axes (although they may very well want to interrogate a back-end dataset that has them). I wonder how this would interact with the behaviour of an API like EDR?

@chris-little
Copy link
Contributor Author

chris-little commented Feb 22, 2022

@jonblower I am happy for multiple time axes to be deferred to a later version.
As for the API-EDR, there is a presumption that there are 4 coordinates for any data (x,y,z,t), but we are adding to the maintenance release next week the support for the possiblity of further 'coordinate' axes, consistent with the other OGC APIs such as API-Coverage.

@chris-little chris-little changed the title How should we process the 20 outstanding CoverageJSON issues How should we process the 22 outstanding CoverageJSON issues Feb 23, 2022
@chris-little
Copy link
Contributor Author

@letmaik @lewismc @jonblower @jerstlouis After Maik's helpful comments, I will create new issues in this repo for the outstanding topics.

@jonblower
Copy link
Contributor

Do the issues have to be moved here, or could we direct people to the original repo? I'm just a bit worried about duplication. Perhaps we could discuss on the call before moving?

@chris-little
Copy link
Contributor Author

@jonblower I have copied , so people can pursue issues in either place, unless we decide otherwise. Let us discuss and close issues and leave their counterparts open, as we agree.
I am keen that the CoverageJSON repo does have a full recognition of the issues with the format, even if discussions take place elsewhere.

@chris-little
Copy link
Contributor Author

2022-03-23 agreed to close as all relevant issues now have a current entry in this repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants