Skip to content

Commit 5a0233b

Browse files
committed
Update documentation
1 parent 320eef9 commit 5a0233b

File tree

7 files changed

+143
-171
lines changed

7 files changed

+143
-171
lines changed

README.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,10 +94,12 @@ Once the needed modules are loaded, you can regularly install via `pip` using th
9494
python -m pip install .
9595
```
9696

97+
Notice that, under the hood, a couple of complex things are going on: the `C++` functions are compiled to a shared library using `pybind11` (automatically taking care of the dependence on `Eigen`), then the library is placed in the right folder and suitable paths are updated, so that the `Python` code is automatically able to import `C++` utils.
98+
9799
## Additional note
98100
There is the possibility that you may need to write the following command:
99101
```bash
100102
conda install -c conda-forge gxx
101103
```
102104
Notice that you should first try *without* using that command, as the normal installation is expected to work.
103-
This is likely a consequence of the fact that we are using a `setup.py` file in order to automatically compile a `C++` file when `pip install .` is executed.
105+
This is likely a consequence of the fact that we are using a `setup.py` file in order to automatically compile a `C++` file when `pip install .` is executed.

docs/Documentation.ipynb

Lines changed: 127 additions & 165 deletions
Large diffs are not rendered by default.

docs/pyclassify.rst

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,3 +27,10 @@ Submodule: helpers_secular
2727
:undoc-members:
2828
:show-inheritance:
2929

30+
Submodule: parallel_tridiag_eigen
31+
================
32+
33+
.. automodule:: pyclassify.parallel_tridiag_eigen
34+
:members:
35+
:undoc-members:
36+
:show-inheritance:

scripts/mpi_running.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
from pyclassify.eigenvalues import Lanczos_PRO
1111

1212

13-
seed = 84
13+
seed = 10
1414
np.random.seed(seed)
1515

1616

scripts/profiling_memory_and_time.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
from mpi4py import MPI
2121

2222

23-
seed = 84
23+
seed = 10
2424
np.random.seed(seed)
2525

2626

@@ -111,7 +111,8 @@
111111
delta_t_parallel = time_after_parallel - time_before_parallel
112112

113113
total_mem_children = comm.reduce(delta_mem, op=MPI.SUM, root=0)
114-
total_time_children = comm.reduce(delta_t_parallel, op=MPI.SUM, root=0)
114+
# total_time_children = comm.reduce(delta_t_parallel, op=MPI.SUM, root=0) #only if we want the sum!
115+
total_time_children = delta_t_parallel
115116

116117
# Collect the information across all ranks
117118
if rank == 0:

shell/submit.sbatch

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ conda init
3232
conda activate devtools_scicomp
3333

3434
# Ranges over which we iterate
35-
n_processes=(1 2 4 8)
35+
n_processes=(2 4 8)
3636
matrix_sizes=(10 50 100 500 1000)
3737

3838
last_dim="${matrix_sizes[-1]}"

shell/submit.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
#!/bin/bash
22

33
# Ranges over which we iterate
4-
n_processes=(1 2 4 8)
4+
n_processes=(2 4 8)
55
matrix_sizes=(10 50 100 500 1000 1500)
66

77
last_dim="${matrix_sizes[-1]}"

0 commit comments

Comments
 (0)