Skip to content

Filtering results using filter library#128

Merged
FranckLecuyer merged 7 commits intomainfrom
filtering_results_using_filter_library
Nov 18, 2025
Merged

Filtering results using filter library#128
FranckLecuyer merged 7 commits intomainfrom
filtering_results_using_filter_library

Conversation

@FranckLecuyer
Copy link
Contributor

@FranckLecuyer FranckLecuyer commented Nov 7, 2025

PR Summary

Using the code in the filter library for the results filtering (linked to PR : gridsuite/computation#13)

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>
Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>
@Meklo
Copy link
Contributor

Meklo commented Nov 12, 2025

FilterService doesn't extends AbstractFilterService so it has a lot of redundant code, shouldn't it be cleaned up by the way ?

…Service class

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>
…ng_filter_library

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

# Conflicts:
#	src/main/java/org/gridsuite/voltageinit/server/service/parameters/FilterService.java
@FranckLecuyer FranckLecuyer force-pushed the filtering_results_using_filter_library branch from a412f1a to 8b5a0a6 Compare November 12, 2025 15:40
…Filters

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>
FranckLecuyer and others added 2 commits November 17, 2025 18:00
Signed-off-by: FranckLecuyer <47824306+FranckLecuyer@users.noreply.github.com>
Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>
@sonarqubecloud
Copy link

@FranckLecuyer FranckLecuyer merged commit 78ffddb into main Nov 18, 2025
4 checks passed
@FranckLecuyer FranckLecuyer deleted the filtering_results_using_filter_library branch November 18, 2025 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants