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

Remove-NXroot-inheritance #1505

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Commits on Nov 4, 2024

  1. Remove inheritance from NXobject

    When group classes that are valid in any base class are added to the NXobject base class, it is important to exclude NXroot to enforce the rule that only NXentry groups are permitted.
    rayosborn committed Nov 4, 2024
    Configuration menu
    Copy the full SHA
    cc82339 View commit details
    Browse the repository at this point in the history
  2. Remove NX_class as an attribute from NXroot

    This is an anomalous entry that is the only example in the entire repository where the mechanism for defining classes is conflated with the contents of that class. Its presence here implies that the attribute is required, whereas it is completely redundant.
    rayosborn committed Nov 4, 2024
    Configuration menu
    Copy the full SHA
    053b8f6 View commit details
    Browse the repository at this point in the history

Commits on Nov 6, 2024

  1. Imrprove the documentation

    The NXroot base class is unique in not applying to an HDF5 group, but to the root level of the NeXus file itself.
    rayosborn committed Nov 6, 2024
    Configuration menu
    Copy the full SHA
    d3032ee View commit details
    Browse the repository at this point in the history

Commits on Nov 7, 2024

  1. Update the documentation to refer to the root level.

    This also adds fields to the permitted non-NeXus objects at the root level.
    rayosborn committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    9bdd4f8 View commit details
    Browse the repository at this point in the history