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

chainedToPrev / chainedToNext still present and should be removed #3

Open
grahambell opened this issue Jul 16, 2014 · 3 comments
Open

Comments

@grahambell
Copy link
Contributor

These elements are still present in some places, such as the UKIRT libraries, despite having been removed from the schema in 508059e. (The outcome is that if you make an observation based on the UKIRT UFTI library, for example, it doesn't validate.)

@timj
Copy link
Contributor

timj commented Jul 16, 2014

Was it removed because the corresponding feature was removed from UFTI or because of an error? (I have to say that I don't think chainedToPrev was ever used by the OMP even if it was used in the previous ORAC system).

@grahambell
Copy link
Contributor Author

That commit mentions 4450fbd which removed them from the OT saying "they haven't been used in years". I don't see chainedTo in the OMP or QT except for in test files. Saving my observation made the OT rewrite the XML without these elements and then it validated, but they still ought to be removed from the libraries.

@timj timj changed the title chainedToPrev / chainedToNext still present chainedToPrev / chainedToNext still present and should be removed Jul 17, 2014
@timj
Copy link
Contributor

timj commented Jul 17, 2014

Agreed. I've updated the title of this issue to make that clearer (I initially thought the issue related to an error in the schema).

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

No branches or pull requests

2 participants