Skip to content

Update time imputation documentation - #42

Merged
irm-codebase merged 1 commit into
modelblocks-org:mainfrom
ddahawkins-TUDelft:update-impute-time-documentation
Aug 7, 2026
Merged

Update time imputation documentation#42
irm-codebase merged 1 commit into
modelblocks-org:mainfrom
ddahawkins-TUDelft:update-impute-time-documentation

Conversation

@ddahawkins-TUDelft

Copy link
Copy Markdown
Collaborator

Fixes #

Summary of changes in this pull request

  • Updated README to describe the new time imputation methods including a representative figure
  • Updated config/README to describe the full argument set

Reviewer checklist

  • There are no pip dependencies in the module's environment files (workflow/envs/).
  • All rules use pathvars (e.g., <results>) in their inputs and outputs.
  • The integration test-suite is successful, including:
    • pre-commit.ci tests pass.
    • tests pass for all relevant OS configurations (linux, osx, windows).
  • Module documentation is up-to-date, including:
    • INTERFACE.yaml mentions all relevant pathvars and wildcards.
    • README.md describes how to use the module and has the necessary citations.

Comment thread README.md

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

If we add more methods we might need a sub-section describing each one. But for now this looks good enough to communicate what the module is doing transparently.

@irm-codebase
irm-codebase merged commit 7dc2f2c into modelblocks-org:main Aug 7, 2026
4 checks passed
@irm-codebase

Copy link
Copy Markdown
Collaborator

@all-contributors please add @ddahawkins-TUDelft for docs

@allcontributors

Copy link
Copy Markdown
Contributor

@irm-codebase

I've put up a pull request to add @ddahawkins-TUDelft! 🎉

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