Smoking variable updates#152
Closed
DougManuel wants to merge 1 commit intov3.0.0-validation-infrastructurefrom
Closed
Smoking variable updates#152DougManuel wants to merge 1 commit intov3.0.0-validation-infrastructurefrom
DougManuel wants to merge 1 commit intov3.0.0-validation-infrastructurefrom
Conversation
DatabaseStart to _m suffix - removed duplicates and typos. Fixed variableStart typos Standard units capitalization Added description and notes fields Updated version to 3.0.0-alpha Updated lastUpdated to 2025-10-26 Added detailed reviewNotes (will be versionNotes in schema PR) Cleared ICES.confirmation fields
Contributor
Author
CSV Standardization and validation@yulric is working on If you want to use them to verify the CSV formatting and structure: # Source the csv-utils (from feature/csv-standardisation-updates branch)
source("R/csv-utils.R")
# Validate the updated variables.csv
result <- standardise_csv("inst/extdata/variables.csv",
collaboration = FALSE,
validate_only = TRUE)
# Check results
print(result$valid) # Should be TRUE
print(result$issues_remaining) # Should be empty
Clean diff achievement
The preserve_column_order = TRUE parameter was used during preparation to ensure minimal diff noise (we have a different column order in our metadata yaml):
49 smoking variables updated (actual content changes)
No column reordering (would have shown 378 lines changed)
Only substantive changes are visible in the diff |
Contributor
Author
|
Closing this PR as the work has been superseded by PR #163 (v3-smoking branch). The original monolithic smoking.R (1,853 lines) has been reorganized into modular files with better architecture:
All functions from this branch have equivalents in #163, which also includes additional variables and more comprehensive tests. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR contains updates to just
variables.csvand just for the smoking variables. I will add the updates to this PR forvariable_details.csvshortly (hopefully).Additional changes to smoking variables that are a lower priority and will be added to a later PR.
Changes to
variables.csv(this PR) include:notesanddescriptionfor all these variables.reviewNotesICES.confirmationfields. I don't think these are needed as we move toward a penultimate review.Observations.MDas these are @mdiaspar notes and she may need them for her review.