- Sep 05, 2023
-
-
Lionel GUEZ authored
-
- Jul 13, 2023
-
-
Lionel GUEZ authored
-
Lionel GUEZ authored
We now want the default behaviour to be renumber. `SHPC_class` expects sequential eddy indices from 1.
-
Lionel GUEZ authored
-
- Jul 12, 2023
-
-
Lionel GUEZ authored
-
Lionel GUEZ authored
Instead of using old result from Matlab. And run on cyclones instead of anticyclones, why not, just one less entry in the namelist.
-
Lionel GUEZ authored
-
- Jun 23, 2023
-
-
Lionel GUEZ authored
-
Lionel GUEZ authored
Because it can be created by analysis scripts.
-
Lionel GUEZ authored
-
Lionel GUEZ authored
The corresponding modules are not used in the functions.
-
Lionel GUEZ authored
-
Lionel GUEZ authored
-
Lionel GUEZ authored
-
Lionel GUEZ authored
This was polluting standard output. So we have to add the shapefile path when it is not found.
-
Lionel GUEZ authored
OSError is more specific than SystemExit. It is better to have a more specific exception when we want to catch it.
-
Lionel GUEZ authored
Invert order of dummy arguments orientation and layer of method `get_reader`. Take the same order than in the path. `orientation` was at the end because, at first, in commit d858fb72, it was an optional argument.
-
Lionel GUEZ authored
It must be the same than the one used for other input files of the test.
-
Lionel GUEZ authored
-
Lionel GUEZ authored
Remove input file `outside_points_get_1_outerm.csv`. Use result from test `Region_1_20454_cyclone_3` instead. There is one less point in the file `outside_points.csv` from the test `Region_1_20454_cyclone_3` but this does not change the results of the two impacted tests: `Get_1_outerm` and `Extr_proj`.
-
Lionel GUEZ authored
The only difference in the test specification of `Get_1_outerm_noise_2` with test `Get_1_outerm` is the input file `outside_points.csv`. This difference in specification leads to no difference in results of the two tests. The input file `outside_points.csv` for `Get_1_outerm` is problematic because it has an extra point at (9.375, -35.857) which corresponds to a cyclone for an old version of SSH data. So we are going to change the input file of `Get_1_outerm` to take `$tests_old_dir/Region_1_20454_cyclone_3/outside_points.csv`, the same one than for `Get_1_outerm_noise_2`, so there is really no use for `Get_1_outerm_noise_2`.
-
Lionel GUEZ authored
-
Lionel GUEZ authored
Remove input file `outside_points_get_1_outerm_noise_2.csv`. Use result from test instead.
-
Lionel GUEZ authored
Motivation: provide file `outside_points.csv` instead of keeping it in `Inst_eddies/Tests/Input/Region_1`.
-
Lionel GUEZ authored
To new version provided by Aviso.
-
Lionel GUEZ authored
There is no significant difference with `Extraction_eddies_region_1_noise`: no impact of `min_amp` on this region. For an impact of `min_amp`, compare tests `Extraction_eddies_region_3` and `Extraction_eddies_region_3_min`.
-
Lionel GUEZ authored
-
Lionel GUEZ authored
-
Lionel GUEZ authored
-
Lionel GUEZ authored
-
Lionel GUEZ authored
-
Lionel GUEZ authored
Bug from commit e84b48a4.
-
Lionel GUEZ authored
It is `min_area` that we need in the code, in `get_1_outerm`. Instead of computing it in the main program and trickling it down to `get_1_outerm`, define it in `config_m`.
-
Lionel GUEZ authored
Create function `plot_grid_points` from code which was in the "script portion" of the module. This new function could be useful when importing the module for interactive work. Also, creating this function clarifies the module.
-
Lionel GUEZ authored
-
Lionel GUEZ authored
This must be the case, since commit f7c5ec74.
-
Lionel GUEZ authored
This is the same as the number of extrema, since commit f7c5ec74.
-
Lionel GUEZ authored
-
Lionel GUEZ authored
It can never happen.
-
Lionel GUEZ authored
We need it to open `e_overestim.txt`. unit was undefined for `i_slice /= - 1`. The bug came from commit af5da9e8.
-