Remove dependencies (#3853)

* nuframe in its own type in UntaggedValue

* Removed eager dataframe from enum

* Dataframe created from list of values

* Corrected order in dataframe columns

* Returned tag from stream collection

* Removed series from dataframe commands

* Arithmetic operators

* forced push

* forced push

* Replace all command

* String commands

* appending operations with dfs

* Testing suite for dataframes

* Unit test for dataframe commands

* improved equality for dataframes

* moving all dataframe operations to protocol

crates/nu-command/Cargo.toml

@@ -99,7 +99,7 @@ zip = { version="0.5.9", optional=true }
 digest = "0.9.0"
 
 [dependencies.polars]
-version = "0.14.7"
+version = "0.14.8"
 optional = true
 features = ["parquet", "json", "random", "pivot", "strings", "is_in"]
 

crates/nu-data/Cargo.toml

@@ -37,13 +37,8 @@ nu-test-support = { version = "0.34.1", path="../nu-test-support" }
 nu-value-ext = { version = "0.34.1", path="../nu-value-ext" }
 nu-ansi-term = { version = "0.34.1", path="../nu-ansi-term" }
 
-[dependencies.polars]
-version = "0.14.7"
-optional = true
-features = ["strings", "checked_arithmetic"]
-
 [target.'cfg(unix)'.dependencies]
 users = "0.11.0"
 
 [features]
-dataframe = ["nu-protocol/dataframe", "polars"]
+dataframe = ["nu-protocol/dataframe"]

crates/nu-data/src/lib.rs

@@ -6,7 +6,4 @@ pub mod primitive;
 pub mod utils;
 pub mod value;
 
-#[cfg(feature = "dataframe")]
-pub mod dataframe;
-
 pub use dict::TaggedListBuilder;

crates/nu-engine/src/evaluate/operator.rs

@@ -5,7 +5,7 @@ use nu_protocol::{Primitive, ShellTypeName, UntaggedValue, Value};
 use std::ops::Not;
 
 #[cfg(feature = "dataframe")]
-use nu_data::dataframe::{compute_between_dataframes, compute_series_single_value};
+use nu_protocol::dataframe::{compute_between_dataframes, compute_series_single_value};
 
 pub fn apply_operator(
     op: Operator,

crates/nu-protocol/Cargo.toml

@@ -32,9 +32,9 @@ serde_yaml = "0.8.16"
 toml = "0.5.8"
 
 [dependencies.polars]
-version = "0.14.7"
+version = "0.14.8"
 optional = true
-features = ["serde", "rows"]
+features = ["serde", "rows", "strings", "checked_arithmetic"]
 
 [features]
 dataframe = ["polars"]

crates/nu-protocol/src/dataframe/compute_between.rs

@@ -1,11 +1,12 @@
 use bigdecimal::BigDecimal;
 use nu_errors::ShellError;
-use nu_protocol::dataframe::{Axis, NuDataFrame};
-use nu_protocol::hir::Operator;
-use nu_protocol::{Primitive, ShellTypeName, UntaggedValue, Value};
 use nu_source::Span;
 use num_traits::ToPrimitive;
 
+use super::{Axis, NuDataFrame};
+use crate::hir::Operator;
+use crate::{Primitive, ShellTypeName, UntaggedValue, Value};
+
 use polars::prelude::{
     BooleanType, ChunkCompare, ChunkedArray, DataType, Float64Type, Int64Type, IntoSeries,
     NumOpsDispatchChecked, PolarsError, Series,

crates/nu-protocol/src/dataframe/mod.rs

@@ -1,7 +1,9 @@
+pub mod compute_between;
 pub mod nu_dataframe;
 pub mod nu_groupby;
 pub mod operations;
 
+pub use compute_between::{compute_between_dataframes, compute_series_single_value};
 pub use nu_dataframe::{Column, NuDataFrame};
 pub use nu_groupby::NuGroupBy;
 pub use operations::Axis;