WeatherValueTimeseriesInterpolation.java #1308

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft

pierrepetersmeier wants to merge 2 commits into dev from pp/#1304-consider-none-equivalent-value-for-missing-data-points-in-weather

Draft

WeatherValueTimeseriesInterpolation.java #1308

Merge branch 'dev' into pp/#1304-consider-none-equivalent-value-for-m…

SonarQubeGithubPRChecks / PowerSystemDataModel Sonarqube Results succeeded May 8, 2025 in 12s

Quality Gate success

Analysis Details

2 Issues

0 Bugs
0 Vulnerabilities
2 Code Smells

Coverage and Duplications

No coverage information (75.90% Estimated after merge)
0.00% Duplicated Code (0.10% Estimated after merge)

Project ID: edu.ie3:PowerSystemDataModel

View in SonarQube

Annotations

Check warning on line 18 in src/main/java/edu/ie3/datamodel/io/factory/timeseries/WeatherValueTimeseriesInterpolation.java

sonarqubegithubprchecks / PowerSystemDataModel Sonarqube Results

src/main/java/edu/ie3/datamodel/io/factory/timeseries/WeatherValueTimeseriesInterpolation.java#L18

Add a private constructor to hide the implicit public one.

Check warning on line 96 in src/main/java/edu/ie3/datamodel/io/factory/timeseries/WeatherValueTimeseriesInterpolation.java

sonarqubegithubprchecks / PowerSystemDataModel Sonarqube Results

src/main/java/edu/ie3/datamodel/io/factory/timeseries/WeatherValueTimeseriesInterpolation.java#L96

Replace this usage of 'Stream.collect(Collectors.toList())' with 'Stream.toList()'

View more details on SonarQubeGithubPRChecks