check_required problème

[Lefort-Antoine]

eplus version 8.0
error with command:
epm = op.Epm.load(source_epm_path, check_required=False)

object causing trouble:

RunPeriod, , !- Name 1, !- Begin Month 1, !- Begin Day of Month 12, !- End Month 31, !- End Day of Month Sunday, !- Day of Week for Start Day No, !- Use Weather File Holidays and Special Days No, !- Use Weather File Daylight Saving Period No, !- Apply Weekend Holiday Rule Yes, !- Use Weather File Rain Indicators Yes; !- Use Weather File Snow Indicators

Error
File "D:/Code/Projects/Transverses/global_tests_ashrea/run_ashrea_references.py", line 201, in simulate_obat epm = op.Epm.load(source_epm_path, check_required=False) File "D:\Code\Bibliotheque\develop\oplus\oplus\epm\epm.py", line 309, in load idd_or_version=idd_or_version File "D:\Code\Bibliotheque\develop\oplus\oplus\epm\epm.py", line 345, in from_idf idd_or_version=idd_or_version File "D:\Code\Bibliotheque\develop\oplus\oplus\epm\epm.py", line 147, in _create_from_buffer_or_path idd_or_version=idd_or_version File "D:\Code\Bibliotheque\develop\oplus\oplus\epm\epm.py", line 123, in __init__ self._dev_populate_from_json_data(json_data) File "D:\Code\Bibliotheque\develop\oplus\oplus\epm\epm.py", line 182, in _dev_populate_from_json_data records = table._dev_add_inert(json_data_records) File "D:\Code\Bibliotheque\develop\oplus\oplus\epm\table.py", line 99, in _dev_add_inert data=r_data File "D:\Code\Bibliotheque\develop\oplus\oplus\epm\record.py", line 47, in __init__ self._update_inert(data) File "D:\Code\Bibliotheque\develop\oplus\oplus\epm\record.py", line 64, in _update_inert self._update_value_inert(k, v) File "D:\Code\Bibliotheque\develop\oplus\oplus\epm\record.py", line 132, in _update_value_inert self._dev_set_none_without_unregistering(index, check_not_required=False) File "D:\Code\Bibliotheque\develop\oplus\oplus\epm\record.py", line 153, in _dev_set_none_without_unregistering f"Field is required (it is a pk). {field_descriptor.get_error_location_message()}") oplus.exceptions.FieldValidationError: Field is required (it is a pk). Table: RunPeriod, index: 0, ref: name.

[geoffroy-destaintot]

The problem is that this field is not just required, it is a primary key, which means it can be used to point on records from other records.
If we bypass check_required here, links management system will break, which is not acceptable.

@Lefort-Antoine
Possible solutions :

• don't allow empty primary keys (even if check_required == False), which is the case actually. Your idf would therefore remain unvalid. To be able to read it, a name must be set to runPeriod before loading the file by opyplus. Where does your idf come from ?
• if check_required == False and primary key is missing we could set a default value ('no_name'?), which would ensure a primary key (would work if only one primary key is missing ; >= 2 missing primary keys would be refused because of duplicates). Given idf could be read, but would be slightly modified on load.
• other ideas ? ...