Merge pull request #985 from MolSSI/fix_dsview_include

Making dataset views should always include required record fields

Author: bennybp

qcfractal/qcfractal/components/dataset_processing/views.py

@@ -12,7 +12,7 @@
 from qcportal.cache import DatasetCache
 from qcportal.dataset_models import BaseDataset
 from qcportal.record_models import RecordStatusEnum, BaseRecord
-from qcportal.utils import chunk_iterable
+from qcportal.utils import chunk_iterable, make_list
 
 if TYPE_CHECKING:
     from sqlalchemy.orm.session import Session
@@ -129,6 +129,10 @@ def create_view_file(
         children_ids = socket.records.get_children_ids(session, record_ids)
         all_ids |= set(children_ids)
 
+    # Always include the base record information
+    if include is not None:
+        include = make_list(include) + ["*"]
+
     ############################################################################
     # Determine the record types of all the ids (top-level and children if desired)
     ############################################################################