Builder Sequential Fix

[supersergiy]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (8e6a908) to head (2bec508).
Report is 2 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff            @@
##              main      #890   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          142       142           
  Lines         6100      6102    +2     
=========================================
+ Hits          6100      6102    +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[nkemnitz]

Thanks for fixing that one!
I think disallowing parallel build for nontrivial objects broke coverage for the parallel part of a non-parallel build? Or is that code now just unreachable?

zetta_utils/zetta_utils/builder/building.py

Lines 154 to 161 in 935d281

	for obj in stage.parallel_part:
	obj_result = _build_object(
	fn=obj.fn,
	kwargs=obj.kwargs,
	spec=obj.spec,
	name_prefix=obj.name_prefix,
	)
	_process_result(obj, obj_result)

[nkemnitz]

Suggested change

	def file_io_funcitons():
	def file_io_functions():

[supersergiy]

Fixed! Indeed, there is no more parallel_part in sequential stages, as the parallel part doesn't respect list order.