Glassvision/IfcOpenShell
1
0
Fork 0
mirror of https://github.com/IfcOpenShell/IfcOpenShell.git synced 2026-08-11 18:16:40 +00:00
Code Issues Packages Projects Releases Wiki Activity

Revert ddd7dff75e

Browse Source 
I guess after all it's not very reliable idea since it's possible to have a cost schedule without quantities (even non-SoR) and it's giving users false assumptions that they don't need to worry about "Is Schedule of Rates" flag in some cases.
This commit is contained in:
Andrej730
2025-04-18 18:41:53 +05:00
parent 1e30eae1d7
commit 93a4f4980a
1 changed files with 3 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ifc5d/ifc5d/csv2ifc.py
+3 -12
View File
@@ -108,7 +108,7 @@ class Csv2Ifc:
parents: dict[int, CostItem] = {}
locale.setlocale(locale.LC_ALL, "") # set the system locale
non_sor_fields = {"Property", "Query"}
non_sor_fields = {"Quantity", "Property", "Query"}
# TODO: 25-04-17 Deprecated 0 indices, should fully remove later.
min_index = None
@@ -126,9 +126,7 @@ class Csv2Ifc:
self.has_categories = False
# validate header
mandatory_fields = {"Name", "Quantity", "Unit"}
if not self.is_schedule_of_rates:
mandatory_fields.update(non_sor_fields)
mandatory_fields = {"Name", "Unit"}
available_fields = set(self.headers.keys())
missing_fields = mandatory_fields - available_fields
@@ -142,14 +140,7 @@ class Csv2Ifc:
missing_fields.remove("Name")
if missing_fields:
if missing_fields == non_sor_fields and not self.is_schedule_of_rates:
self.is_schedule_of_rates = True
print(
"WARNING. Assumed the imported cost schedule is a schedule of rates "
f"because the following fields are missing: {', '.join(non_sor_fields)}."
)
else:
raise Exception(f"Missing mandatory fields in CSV header: {', '.join(missing_fields)}")
raise Exception(f"Missing mandatory fields in CSV header: {', '.join(missing_fields)}")
continue
cost_data = self.get_row_cost_data(row)