InseeFrLab / InseeFrLab/benchmark_spatial_interpolation

Feedback on the part on results

Open
#10 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Jupyter Notebook
Stars
1
Forks
1
PR merge metrics
No merged PRs in 30d

Description

I recompiled the report myself, and I wonder if I did something wrong, because the results section is inconsistent and contains many typos. Is it a draft? Did you forget to push commits?

In its present state, it requires major changes. Some of them are:

- Section numbers are wrong (double numbers);
- The legend of the "Small Datasets: Efficiency vs. Accuracy Trade-off" figure is incomplete: what do shapes stand for?
- This legend is inconsistent with the text.
- The figure "Impact of CR on Large Datasets" makes no sense at all.
- A figure appears to be missing in the clean versus noisy analysis.
- The discussion feels like chatGPT and is not in line with the results (it mentions a heatmap that is not there any more).

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.