ivoa-std / VOTable

VOTable Format Definition
4 stars 15 forks source link

Review Appendix A, "Possible VOTable extensions" #53

Open tomdonaldson opened 9 months ago

tomdonaldson commented 9 months ago

As mentioned in #51, Appendix A ("Possible VOTable extensions") has a number of things that are not consistent with the current state of VOTable, other standards or intended uses.

As this appendix was somewhat speculative, perhaps it could be reduce to listing common patterns that have evolved with VOTable, or indeed removed entirely. For parts that are still considered valuable, we should certainly update the examples to be compliant with current standards and guidance.

msdemlei commented 9 months ago

On Fri, Nov 24, 2023 at 10:45:13AM -0800, Tom wrote:

As this appendix was somewhat speculative, perhaps it could be reduce to listing common patterns that have evolved with VOTable, or indeed removed entirely. For parts that are still considered

If you ask me: Let's just remove it. It's been a constant source of confusion. Also, we now have Notes established, so if folks want to present clever (if non-normative) ways to use VOTable, that's where they can do it.

mbtaylor commented 9 months ago

Removing it would get my support.

tomdonaldson commented 9 months ago

I agree with removing it. It's very tempting to just rip it out now for v1.5, but I'm not sure the best way to change the several references to it in the text (and don't really have a moment to try).

If anyone feels like it would actually be a clean removal and has time to make a PR soon, I would support it, but am also comfortable waiting for v1.6.

Bonnarel commented 9 months ago

Le 29/11/2023 à 15:07, Tom a écrit :

I agree with removing it. It's very tempting to just rip it out now for v1.5, but I'm not sure the best way to change the several references to it in the text (and don't really have a moment to try).

If anyone feels like it would actually be a clean removal and has time to make a PR soon, I would support it, but am also comfortable waiting for v1.6.

— Reply to this email directly, view it on GitHub https://github.com/ivoa-std/VOTable/issues/53#issuecomment-1831964752, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMP5LTGEZKGSZLECTBBYVKLYG46T7AVCNFSM6AAAAAA7ZO63FWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZRHE3DINZVGI. You are receiving this because you are subscribed to this thread.Message ID: @.***>

As you can imagine CDS is not really happy with that. After discussing here, we follow the idea to write an implmentation note instead. Having this change for v1.6 will let us the time to propose one.

Cheers

François

tomdonaldson commented 8 months ago

That sounds like a really good plan, i.e., let's remove Appendix A in v1.6 and replace it with an implementation note with updated guidance.