Skip to content

show how to validate an XML file#1497

Merged
prjemian merged 2 commits intomainfrom
1380-how-to-validate-XML-files
Oct 17, 2024
Merged

show how to validate an XML file#1497
prjemian merged 2 commits intomainfrom
1380-how-to-validate-XML-files

Conversation

@prjemian
Copy link
Copy Markdown
Contributor

@prjemian prjemian added this to the NXDL 2025 milestone Oct 15, 2024
@prjemian prjemian requested a review from a team October 15, 2024 16:32
@prjemian
Copy link
Copy Markdown
Contributor Author

New documentation looks like this:
image

Comment thread manual/source/examples/epics/index.rst
Copy link
Copy Markdown
Contributor

@PeterC-DLS PeterC-DLS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

BTW layout.xml doesn't have schema metadata

@prjemian
Copy link
Copy Markdown
Contributor Author

prjemian commented Oct 16, 2024 via email

@PeterC-DLS
Copy link
Copy Markdown
Contributor

Can you explain?

$ head -3 manual/source/examples/epics/layout.xml 
<?xml version="1.0" standalone="no" ?>
<hdf5_layout>
  <group name="entry">

cf

$ head -6 manual/source/examples/epics/attributes.xml
<?xml version="1.0" standalone="no" ?>
<!-- Attributes -->
<Attributes
  xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
  xsi:noNamespaceSchemaLocation="https://github.com/areaDetector/ADCore/blob/master/iocBoot/NDAttributes.xsd"
  >                                            

So there's a reference to the schema in the latter file

@prjemian
Copy link
Copy Markdown
Contributor Author

The metadata about XML Schema location is not necessary to validate the XML layout files. The schema location was not added in ADCore since it was mentioned as an addition for layout files.

@PeterC-DLS
Copy link
Copy Markdown
Contributor

Sure, I was just remarking on its absence as it's inconsistent.

@prjemian prjemian merged commit 979ec79 into main Oct 17, 2024
@prjemian prjemian deleted the 1380-how-to-validate-XML-files branch October 17, 2024 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

validating XML files for area detector HDF5 writer

2 participants