Skip to content

Commit

Permalink
Remove the isBuilt check for now
Browse files Browse the repository at this point in the history
  • Loading branch information
Matevz Morato committed May 7, 2024
1 parent 667764b commit 2fa7670
Showing 1 changed file with 7 additions and 5 deletions.
12 changes: 7 additions & 5 deletions src/pipeline/Pipeline.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -917,11 +917,13 @@ void PipelineImpl::build() {
void PipelineImpl::start() {
std::lock_guard<std::mutex> lock(stateMtx);
// TODO(themarpe) - add mutex and set running up ahead
for(const auto& node : getAllNodes()) {
if (node->needsBuild()) {
throw std::runtime_error(fmt::format("Node '{}' was not built", node->getName()));
}
}

// TODO(Morato) - add back in when more nodes are tested
// for(const auto& node : getAllNodes()) {
// if (node->needsBuild()) {
// throw std::runtime_error(fmt::format("Node '{}' was not built", node->getName()));
// }
// }

// Implicitly build (if not already)
build();
Expand Down

0 comments on commit 2fa7670

Please sign in to comment.