use cobre_core::EntityId;
use parquet::arrow::arrow_reader::ParquetRecordBatchReaderBuilder;
use std::fs::File;
use std::path::Path;
use crate::LoadError;
use crate::parquet_helpers::{extract_required_float64, extract_required_int32};
#[derive(Debug, Clone, PartialEq)]
pub struct InflowArCoefficientRow {
pub hydro_id: EntityId,
pub stage_id: i32,
pub lag: i32,
pub coefficient: f64,
}
pub fn parse_inflow_ar_coefficients(path: &Path) -> Result<Vec<InflowArCoefficientRow>, LoadError> {
let file = File::open(path).map_err(|e| LoadError::io(path, e))?;
let builder = ParquetRecordBatchReaderBuilder::try_new(file)
.map_err(|e| LoadError::parse(path, e.to_string()))?;
let reader = builder
.build()
.map_err(|e| LoadError::parse(path, e.to_string()))?;
let mut rows: Vec<InflowArCoefficientRow> = Vec::new();
for batch_result in reader {
let batch = batch_result.map_err(|e| LoadError::parse(path, e.to_string()))?;
let hydro_id_col = extract_required_int32(&batch, "hydro_id", path)?;
let stage_id_col = extract_required_int32(&batch, "stage_id", path)?;
let lag_col = extract_required_int32(&batch, "lag", path)?;
let coefficient_col = extract_required_float64(&batch, "coefficient", path)?;
let n = batch.num_rows();
let base_idx = rows.len();
rows.reserve(n);
for i in 0..n {
let row_idx = base_idx + i;
let hydro_id = EntityId::from(hydro_id_col.value(i));
let stage_id = stage_id_col.value(i);
let lag = lag_col.value(i);
let coefficient = coefficient_col.value(i);
if lag < 1 {
return Err(LoadError::SchemaError {
path: path.to_path_buf(),
field: format!("inflow_ar_coefficients[{row_idx}].lag"),
message: format!("lag must be >= 1 (1-based), got {lag}"),
});
}
rows.push(InflowArCoefficientRow {
hydro_id,
stage_id,
lag,
coefficient,
});
}
}
rows.sort_by(|a, b| {
a.hydro_id
.0
.cmp(&b.hydro_id.0)
.then_with(|| a.stage_id.cmp(&b.stage_id))
.then_with(|| a.lag.cmp(&b.lag))
});
Ok(rows)
}
#[cfg(test)]
#[allow(
clippy::doc_markdown,
clippy::expect_used,
clippy::panic,
clippy::too_many_lines,
clippy::unwrap_used
)]
mod tests {
use super::*;
use arrow::array::{Float64Array, Int32Array};
use arrow::datatypes::{DataType, Field, Schema};
use arrow::record_batch::RecordBatch;
use parquet::arrow::ArrowWriter;
use std::sync::Arc;
use tempfile::NamedTempFile;
fn schema() -> Arc<Schema> {
Arc::new(Schema::new(vec![
Field::new("hydro_id", DataType::Int32, false),
Field::new("stage_id", DataType::Int32, false),
Field::new("lag", DataType::Int32, false),
Field::new("coefficient", DataType::Float64, false),
]))
}
fn schema_with_extra_column() -> Arc<Schema> {
Arc::new(Schema::new(vec![
Field::new("hydro_id", DataType::Int32, false),
Field::new("stage_id", DataType::Int32, false),
Field::new("lag", DataType::Int32, false),
Field::new("coefficient", DataType::Float64, false),
Field::new("extra", DataType::Float64, false),
]))
}
fn write_parquet(batch: &RecordBatch) -> NamedTempFile {
let tmp = NamedTempFile::new().expect("tempfile");
let mut writer = ArrowWriter::try_new(tmp.reopen().expect("reopen"), batch.schema(), None)
.expect("ArrowWriter");
writer.write(batch).expect("write batch");
writer.close().expect("close writer");
tmp
}
fn make_batch(
hydro_ids: &[i32],
stage_ids: &[i32],
lags: &[i32],
coefficients: &[f64],
) -> RecordBatch {
RecordBatch::try_new(
schema(),
vec![
Arc::new(Int32Array::from(hydro_ids.to_vec())),
Arc::new(Int32Array::from(stage_ids.to_vec())),
Arc::new(Int32Array::from(lags.to_vec())),
Arc::new(Float64Array::from(coefficients.to_vec())),
],
)
.expect("valid batch")
}
fn make_batch_with_extra(
hydro_ids: &[i32],
stage_ids: &[i32],
lags: &[i32],
coefficients: &[f64],
extras: &[f64],
) -> RecordBatch {
RecordBatch::try_new(
schema_with_extra_column(),
vec![
Arc::new(Int32Array::from(hydro_ids.to_vec())),
Arc::new(Int32Array::from(stage_ids.to_vec())),
Arc::new(Int32Array::from(lags.to_vec())),
Arc::new(Float64Array::from(coefficients.to_vec())),
Arc::new(Float64Array::from(extras.to_vec())),
],
)
.expect("valid batch with extra column")
}
#[test]
fn test_valid_6_rows_sorted_by_hydro_stage_lag() {
let batch = make_batch(
&[2, 2, 2, 1, 1, 1],
&[0, 0, 0, 0, 0, 0],
&[3, 2, 1, 2, 3, 1],
&[0.1, 0.2, 0.3, 0.4, 0.5, 0.6],
);
let tmp = write_parquet(&batch);
let rows = parse_inflow_ar_coefficients(tmp.path()).unwrap();
assert_eq!(rows.len(), 6);
assert_eq!(rows[0].hydro_id, EntityId::from(1));
assert_eq!(rows[0].lag, 1);
assert_eq!(rows[1].hydro_id, EntityId::from(1));
assert_eq!(rows[1].lag, 2);
assert_eq!(rows[2].hydro_id, EntityId::from(1));
assert_eq!(rows[2].lag, 3);
assert_eq!(rows[3].hydro_id, EntityId::from(2));
assert_eq!(rows[3].lag, 1);
assert_eq!(rows[4].hydro_id, EntityId::from(2));
assert_eq!(rows[4].lag, 2);
assert_eq!(rows[5].hydro_id, EntityId::from(2));
assert_eq!(rows[5].lag, 3);
}
#[test]
fn test_lag_zero_is_schema_error() {
let batch = make_batch(&[1], &[0], &[0], &[0.45]);
let tmp = write_parquet(&batch);
let err = parse_inflow_ar_coefficients(tmp.path()).unwrap_err();
match &err {
LoadError::SchemaError { field, message, .. } => {
assert!(
field.contains("lag"),
"field should contain 'lag', got: {field}"
);
assert!(
message.contains('1'),
"message should mention >= 1, got: {message}"
);
}
other => panic!("expected SchemaError, got: {other:?}"),
}
}
#[test]
fn test_missing_coefficient_column() {
let schema_no_coeff = Arc::new(Schema::new(vec![
Field::new("hydro_id", DataType::Int32, false),
Field::new("stage_id", DataType::Int32, false),
Field::new("lag", DataType::Int32, false),
]));
let batch = RecordBatch::try_new(
schema_no_coeff,
vec![
Arc::new(Int32Array::from(vec![1_i32])),
Arc::new(Int32Array::from(vec![0_i32])),
Arc::new(Int32Array::from(vec![1_i32])),
],
)
.unwrap();
let tmp = write_parquet(&batch);
let err = parse_inflow_ar_coefficients(tmp.path()).unwrap_err();
match &err {
LoadError::SchemaError { field, message, .. } => {
assert!(
field.contains("coefficient"),
"field should contain 'coefficient', got: {field}"
);
assert!(
message.contains("missing required column"),
"message should mention missing column, got: {message}"
);
}
other => panic!("expected SchemaError, got: {other:?}"),
}
}
#[test]
fn test_empty_parquet_returns_empty_vec() {
let batch = make_batch(&[], &[], &[], &[]);
let tmp = write_parquet(&batch);
let rows = parse_inflow_ar_coefficients(tmp.path()).unwrap();
assert!(rows.is_empty());
}
#[test]
fn test_coefficient_values_preserved() {
let batch = make_batch(&[42], &[3], &[1], &[0.12345]);
let tmp = write_parquet(&batch);
let rows = parse_inflow_ar_coefficients(tmp.path()).unwrap();
assert_eq!(rows.len(), 1);
let row = &rows[0];
assert_eq!(row.hydro_id, EntityId::from(42));
assert_eq!(row.stage_id, 3);
assert_eq!(row.lag, 1);
assert!((row.coefficient - 0.12345).abs() < 1e-10);
}
#[test]
fn extra_columns_are_ignored() {
let batch = make_batch_with_extra(
&[1, 1, 2],
&[0, 0, 0],
&[1, 2, 1],
&[0.5, 0.2, 0.3],
&[0.85, 0.85, 0.9],
);
let tmp = write_parquet(&batch);
let rows =
parse_inflow_ar_coefficients(tmp.path()).expect("a file with extra columns must parse");
assert_eq!(rows.len(), 3, "all rows must still parse");
}
}