Cleaned up code

bempp · Jan 17, 2025 · 162f6ac · 162f6ac
1 parent 281ed9b
commit 162f6ac
Show file tree

Hide file tree

Showing 8 changed files with 11 additions and 20 deletions.
diff --git a/benches/assembly_benchmark.rs b/benches/assembly_benchmark.rs
@@ -5,7 +5,6 @@ use bempp::laplace::assembler::single_layer;
 use criterion::{black_box, criterion_group, criterion_main, Criterion};
 use ndelement::ciarlet::LagrangeElementFamily;
 use ndelement::types::{Continuity, ReferenceCellType};
-use rlst::IndexLayout;
 
 pub fn assembly_parts_benchmark(c: &mut Criterion) {
     let mut group = c.benchmark_group("assembly");
@@ -28,7 +27,6 @@ pub fn assembly_parts_benchmark(c: &mut Criterion) {
         options.set_batch_size(128);
 
         let assembler = single_layer(&options);
-        let index_layout = IndexLayout::from_local_counts(space.global_size(), &comm);
 
         group.bench_function(
             format!(

diff --git a/examples/laplace_evaluator.rs b/examples/laplace_evaluator.rs
@@ -17,7 +17,7 @@ use rand::SeedableRng;
 use rand_chacha::ChaCha8Rng;
 use rlst::{
     operator::{interface::DistributedArrayVectorSpace, zero_element, Operator},
-    rlst_dynamic_array1, AsApply, Element, MultInto, OperatorBase,
+    rlst_dynamic_array1, AsApply, MultInto, OperatorBase,
 };
 
 fn main() {

diff --git a/examples/neighbour_evaluator.rs b/examples/neighbour_evaluator.rs
@@ -15,7 +15,7 @@ use rand::SeedableRng;
 use rand_chacha::ChaCha8Rng;
 use rlst::{
     operator::{interface::DistributedArrayVectorSpace, zero_element},
-    rlst_dynamic_array2, AsApply, Element, IndexLayout, LinearSpace, RandomAccessMut, RawAccess,
+    rlst_dynamic_array2, AsApply, IndexLayout, RandomAccessMut, RawAccess,
 };
 
 fn main() {

diff --git a/examples/test_green_evaluators.rs b/examples/test_green_evaluators.rs
@@ -9,7 +9,7 @@ use rand::{Rng, SeedableRng};
 use rand_chacha::ChaCha8Rng;
 use rlst::{
     operator::{interface::DistributedArrayVectorSpace, zero_element},
-    rlst_dynamic_array1, AsApply, Element, IndexLayout, LinearSpace, NormedSpace, RawAccessMut,
+    rlst_dynamic_array1, AsApply, IndexLayout, RawAccessMut,
 };
 
 fn main() {

diff --git a/src/boundary_assemblers.rs b/src/boundary_assemblers.rs
@@ -25,9 +25,8 @@ use ndgrid::traits::{Entity, Grid, Topology};
 use ndgrid::types::Ownership;
 use rayon::prelude::*;
 use rlst::{
-    rlst_dynamic_array2, rlst_dynamic_array4, CsrMatrix, DefaultIterator, DistributedCsrMatrix,
-    DynamicArray, IndexLayout, MatrixInverse, RandomAccessMut, RawAccess, RawAccessMut, RlstScalar,
-    Shape,
+    rlst_dynamic_array2, rlst_dynamic_array4, DefaultIterator, DistributedCsrMatrix, DynamicArray,
+    MatrixInverse, RandomAccessMut, RawAccess, RawAccessMut, RlstScalar, Shape,
 };
 use std::collections::HashMap;
 

diff --git a/src/evaluator_tools.rs b/src/evaluator_tools.rs
@@ -1,7 +1,6 @@
 //! Various helper functions to support evaluators.
 
 use std::{
-    cell,
     collections::{HashMap, HashSet},
     marker::PhantomData,
     rc::Rc,
@@ -17,22 +16,22 @@ use ndgrid::{
     types::Ownership,
 };
 
-use rayon::prelude::*;
 use rlst::{
     operator::{
         interface::{
-            distributed_array_vector_space,
             distributed_sparse_operator::DistributedCsrMatrixOperatorImpl,
             DistributedArrayVectorSpace,
         },
-        zero_element, Operator,
+        Operator,
     },
     rlst_array_from_slice2, rlst_dynamic_array1, rlst_dynamic_array2, rlst_dynamic_array3,
     rlst_dynamic_array4, Array, AsApply, DefaultIterator, DistributedCsrMatrix, DistributedVector,
     Element, IndexLayout, OperatorBase, RawAccess, RawAccessMut, RlstScalar, Shape,
     UnsafeRandomAccessByValue, UnsafeRandomAccessMut,
 };
 
+use rayon::prelude::*;
+
 use crate::function::FunctionSpaceTrait;
 
 /// Create a linear operator from the map of a basis to points. The points are sorted by global

diff --git a/src/function.rs b/src/function.rs
@@ -13,7 +13,6 @@ use ndgrid::{traits::Grid, types::Ownership};
 use rlst::{IndexLayout, MatrixInverse, RlstScalar};
 use std::collections::HashMap;
 use std::marker::PhantomData;
-use std::ops::Index;
 use std::rc::Rc;
 
 type DofList = Vec<Vec<usize>>;

diff --git a/src/greens_function_evaluators/kifmm_evaluator.rs b/src/greens_function_evaluators/kifmm_evaluator.rs
@@ -21,15 +21,11 @@ use kifmm::{
     tree::SortKind,
     ChargeHandler, Evaluate, FftFieldTranslation, KiFmm, KiFmmMulti, MultiNodeBuilder,
 };
-use mpi::{
-    ffi::MPI_T_ERR_PVAR_NO_STARTSTOP,
-    traits::{Communicator, Equivalence},
-};
+use mpi::traits::{Communicator, Equivalence};
 use num::Float;
 use rlst::{
-    operator::{interface::DistributedArrayVectorSpace, zero_element},
-    rlst_dynamic_array1, AsApply, Element, IndexLayout, MatrixSvd, OperatorBase, RawAccess,
-    RawAccessMut, RlstScalar,
+    operator::interface::DistributedArrayVectorSpace, rlst_dynamic_array1, AsApply, Element,
+    IndexLayout, MatrixSvd, OperatorBase, RawAccess, RawAccessMut, RlstScalar,
 };
 
 /// This structure instantiates an FMM evaluator.