ThinkR-open / ThinkR-open/datadiff
[chore] Ménage : dev/ dans le tarball, inst/templates mort, imports inutilisés, dépendances superflues, scories diverses
Open
Nobody has claimed this yet.
- Dominant language
- R
- Stars
- 6
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
Contexte
Rollup des constats basse sévérité de l'audit (chacun trivial isolément, à traiter en un ou deux PR de ménage) :
Packaging
dev/versionné et absent de.Rbuildignore→ embarqué dans le tarball (NOTE « non-standard top-level directory ») ;dev/bench/perf_issue4.Rréférence un fichier inexistant (dev/dimensions_tdc_lyon.md) etdevtoolshors DESCRIPTION.inst/templates/rules_template.yaml: référencé nulle part (aucunsystem.filedans le repo) — supprimer ou l'utiliser.
Dépendances/imports
@importFrom dplyr arrange:arrangejamais utilisé ;across/collectimportés mais toujours appelés en forme namespacée.tidyselecten Imports pour un seulall_of(ré-exporté par dplyr) ;statspour un uniquesetNames(remplaçable base) ;DBIutilisé dans utils.R sans garderequireNamespacepropre (implicite via duckdb).
Code
- Littéraux décimaux illisibles :
0.000000001,0.00000000000001→1e-9,1e-14. - Variable de boucle
cmasquantbase::c(tolerance.R, coverage.R, fast_path.R, compare_datasets_from_yaml.R, pointblank_setup.R). tolower(x = x)(preprocessing.R) ; duplication de la liste de classes Arrow entreis_non_local()etis_arrow();fp_epsrecalculé par itération (add_tolerance_columns) ;abs(ref_vals)calculé 2× (compute_tolerance_ok) ;sum(is.na(ok) | !ok)→length(ok) - sum(ok, na.rm = TRUE)(2× plus rapide, 0 alloc).format_key_examples(): formate toutes les clés avant de tronquer à 3 ; paddingas.matrixsur clés mixtes.add_diff_columns(): paramna_equalsans effet sur la sortie (doc à préciser) ; squelette boucle+list2DF+cbinddupliqué 3× dans tolerance.R (composeradd_ok_columns+add_diff_columnsdansadd_tolerance_columns).preprocess_dataframe()lazy : 1-2mutate()par colonne normalisée (anti-pattern O(colonnes) dbplyr éliminé ailleurs en 0.4.8) ; branche Arrow morte sur le pipeline principal (conversion DuckDB en amont).- Rapport gt construit même en session non interactive dans
print.datadiff_report; overflow integer possible danscompute_tolerance_colsur colonnes integer (documenter ou caster). get_col_names(): fallbacknames(x)inatteignable dans le package.
Critères de succès
-
R CMD checksans nouvelle NOTE (dev/ ignoré, imports nettoyés, tidyselect/stats retirés si retenu). - Chaque point traité ou explicitement écarté (wontfix commenté) dans la PR de ménage.
- Suite de tests inchangée et verte ; aucun changement de comportement utilisateur (sauf points documentés).
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with R CMD check, then inspect .Rbuildignore, DESCRIPTION, dev/, inst/templates/rules_template.yaml, and the R files named in the issue. Work through the listed cleanup points and record any explicitly deferred items. Done means no new check NOTE, the test suite remains green, and user-visible behavior is unchanged except where documented.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- r
- Domain
- build-system, tooling
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100