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

XTCE generator scripts #680

Merged
2 changes: 1 addition & 1 deletion imap_processing/cdf/cdf_attribute_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -216,7 +216,7 @@ def load_variable_attributes(self, file_name: str) -> None:
The name of the file to load from self.source_dir.
"""
# Add variable attributes from file_name. Each variable name should have the
# required sub fields as defined in the variable schema.
# Required subfields as defined in the variable schema.
raw_var_attrs = CdfAttributeManager._load_yaml_data(self.source_dir / file_name)
var_attrs = raw_var_attrs.copy()

Expand Down
3 changes: 3 additions & 0 deletions imap_processing/tests/hi/test_hi_l1b.py
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think this was fixed in #681 so shouldn't be added back in here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Haha, yes! sorry about that.

Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
"""Test coverage for imap_processing.hi.l1b.hi_l1b.py"""

import pytest

from imap_processing import imap_module_directory
from imap_processing.hi.l1a.hi_l1a import hi_l1a
from imap_processing.hi.l1b.hi_l1b import hi_l1b
Expand All @@ -19,6 +21,7 @@ def test_hi_l1b_hk():
assert l1b_dataset.attrs["Logical_source"] == "imap_hi_l1b_45sensor-hk"


@pytest.mark.xfail()
def test_hi_l1b_de(create_de_data, tmp_path):
"""Test coverage for imap_processing.hi.hi_l1b.hi_l1b() with
direct events L1A as input"""
Expand Down
Loading
Loading