Skip to content
Snippets Groups Projects
Commit 990f670c authored by Laura Christine Kühle's avatar Laura Christine Kühle
Browse files

Removed completed TODOs.

parent ad0b2977
Branches
No related tags found
No related merge requests found
......@@ -15,26 +15,6 @@ TODO: Ask whether all quadratures depend on freely chosen num_nodes
TODO: Contemplate saving training data for each IC separately
TODO: Contemplate removing TrainingDataGenerator class
TODO: Ask why no fine mesh is used for calculate_coarse_projection()
TODO: Contemplate keeping Theoretical threshold over all degrees (only p)
-> Done (same options as Boxplot)
TODO: Discuss whether Threshold detector is needed
TODO: Contemplate reasoning behind Adjusted-Outer-Fence method -> Done
TODO: Discuss why indexed_coeffs are largest absolute value instead of
mean for Boxplot TC -> Done (give option)
TODO: Discuss why overlapping cells are only introduced for one adjacent
fold (right neighbour, except for last fold) -> Done
TODO: Discuss whether quartiles do correspond to Thea's calculation
-> Done (do not, change)
TODO: Discuss that quartiles do not take overlapping cells into account
-> Done (change)
TODO: Contemplate using pre-implemented quartile functions -> Done (do)
TODO: Contemplate restricting number of overlapping cells to half of fold
length -> Done (Restrict to 1 for now)
TODO: Contemplate whether overlapping cells should be evaluated in
neighboring folds -> Done (no)
TODO: Discuss if folds are valid for all possible num_grid_cells/fold_len
-> Done (enforce)
TODO: Contemplate combining get_cells() and _get_cells() -> Done (do)
Urgent:
TODO: Enforce mesh with 2^n cells
......
......@@ -2,10 +2,6 @@
"""
@author: Laura C. Kühle, Soraya Terrab (sorayaterrab)
TODO: Vectorize _get_cells() in Boxplot method -> Done
TODO: Restructure Boxplot method -> Done
TODO: Introduce lower/upper extreme outliers in Boxplot
(each cell is also checked for neighboring domains if existing) -> Done
TODO: Give option to choose from multiwavelet degrees (first, last or
highest magnitude)
TODO: Change method of calculating quartiles
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment