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

Missing Alice and LORRI Documentation and Regression Tests #27

Open
8 of 19 tasks
acraugh opened this issue May 28, 2024 · 0 comments
Open
8 of 19 tasks

Missing Alice and LORRI Documentation and Regression Tests #27

acraugh opened this issue May 28, 2024 · 0 comments
Assignees

Comments

@acraugh
Copy link
Contributor

acraugh commented May 28, 2024

The Alice and LORRI dictionary schema changes have been merged without the required documentation and regression tests. These need to be completed prior to releasing the dictionary, which in turn needs to happen when the data are published. The Pull Request Check List contains sections on both documentation updates and regression tests needed. Here they are, for convenience:

Documentation

  • Update the Change Log.
  • Add all new classes and attributes to the namespace outline.
  • Document new classes and attributes added to nh:Mission_Parameters in the user documentation.
  • Add examples of new classes and attributes.

In ChangeLog.md

  • List the major changes and additions made to the namespace individually.
  • Minor changes (like typos) may be grouped into a single mention.
  • Remove any changes that were previously documented but deleted from the branch before release.

In detailed/outline.rst

  • Add new attributes and classes to the namespace outline following the existing style.

In index.rst:

  • Add new examples to the table of contents.

In user/user-guide.rst:

  • Add new major subclasses to the content list for nh:Mission_Parameters in the correct order and link it to the following subclass heading.
  • Add a new section for each new major subclass in the correct order following the model of the existing classes (e.g., nh:Observational_Parameters).
  • Add new subsections for all new attributes added to the nh:Mission_Parameters class or any of its major subclasses.

In examples/

  • Add a new label if there is no existing example for a new instrument. Follow the model of the existing Mockup.rst files.
    OR
  • Modify an existing label to show the use of a new class that is specific to an instrument.
  • Supply a partial label to show the use of a new class that would not easily fit into the existing label(s) for the relevant instrument.

Regression Tests

  • Add new regression tests or update existing regression tests to illustrate correct use of new classes (the label used for the example can be re-used here).
  • Add regression tests for critical items enforced by schema constraint (both VALID and FAIL).
  • Add regression tests to demonstrate bug resolution with a reference to the specific issue raised (this may require only one of either VALID or FAIL tests in some cases).
  • Document new regression tests in test/README.md following the pattern for existing tests. Bug fixes should reference the issue number.
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