16299120424

Committed 15 Jul 2025 04:39PM UTC coverage: 74.559% (+0.04%) from 74.522%

Build # 16299120424

Build Type

push

github

Committed by

paulmthompson

Commit Message

try using new instantiation method that is clang compatible

Run Details

33 of 37 new or added lines in 2 files covered. (89.19%)

15245 of 20447 relevant lines covered (74.56%)

2088.0 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

90.91

/src/WhiskerToolbox/DataManager/utils/TableView/columns/Column.cpp

#include "Column.h"
#include "utils/TableView/core/TableView.h"

#include <stdexcept>

template<typename T>
auto Column<T>::getValues(TableView* table) -> const std::vector<T>& {
    if (!isMaterialized()) {
        materialize(table);
    }
    return std::get<std::vector<T>>(m_cache);
}

template<typename T>
void Column<T>::materialize(TableView* table) {
    if (isMaterialized()) {
        return;
    }

    // Get the execution plan for this column's data source
    const auto& plan = table->getExecutionPlanFor(getSourceDependency());
    
    // Compute the column values using the computer
    auto computed = m_computer->compute(plan);
    
    // Store the computed values in the cache
    m_cache = std::move(computed);
}



// Explicit instantiation for commonly used types
template class Column<double>;
template class Column<bool>;
template class Column<int>;
template class Column<std::vector<double>>;
template class Column<std::vector<int>>;
template class Column<std::vector<TimeFrameIndex>>;

1	#include "Column.h"
2	#include "utils/TableView/core/TableView.h"
3
4	#include <stdexcept>
5
6	template<typename T>
7	auto Column<T>::getValues(TableView* table) -> const std::vector<T>& {	11✔
8	if (!isMaterialized()) {	11✔
9	materialize(table);	10✔
10	}
11	return std::get<std::vector<T>>(m_cache);	11✔
12	}
13
14	template<typename T>
15	void Column<T>::materialize(TableView* table) {	10✔
16	if (isMaterialized()) {	10✔
NEW 17	return;	×
18	}
19
20	// Get the execution plan for this column's data source
21	const auto& plan = table->getExecutionPlanFor(getSourceDependency());	10✔
22
23	// Compute the column values using the computer
24	auto computed = m_computer->compute(plan);	10✔
25
26	// Store the computed values in the cache
27	m_cache = std::move(computed);	10✔
28	}	10✔
29
30
31
32	// Explicit instantiation for commonly used types
33	template class Column<double>;
34	template class Column<bool>;
35	template class Column<int>;
36	template class Column<std::vector<double>>;
37	template class Column<std::vector<int>>;
38	template class Column<std::vector<TimeFrameIndex>>;

paulmthompson / WhiskerToolbox / 16299120424

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous