Repository navigation
Fix package load; new randomise() by Jacob van Etten; fewer dependencies - #15
Merged
Merged
Conversation
…Solve, Matrix, RSpectra, methods - randomize() now builds a sequentially balanced incomplete block design (replication schedule, exact A-optimality tie-breaking, hard availability, position balance); old arguments are ignored with a warning - Imports reduced to httr, jsonlite, stats, utils - tests for prefixes, proportions, ncomp = 2 and ignored arguments - old diagnostic script moved to dev/ so R CMD check does not run it - workflow reports R CMD INSTALL errors as annotations on failure Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LpE9sTbGPftSMogW6KpkrT
…vey() The file ended with top-level code that called the API and wrote to a local path at load time, which broke R CMD INSTALL on all platforms. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LpE9sTbGPftSMogW6KpkrT
kauedesousa
changed the title
kauedesousa
marked this pull request as ready for review
October 31, 2026 14:00
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.
Follow-up to #14 (merged before these commits landed).
Fix: the package could not be installed.
R/getSurvey.Rended with scratch code (an API key, a project, andwrite.csv()to a local Mac path) that ran at load time. This is the "unable to load R code" failure on #14, and it also affects master now. Removed;getSurvey()is now exported and documented.New
randomise()algorithm (Jacob van Etten)ridge,polish,schedule; old arguments such ascompare ignored with a warning. Designs differ from earlier versions for the same seed.lpSolve,Matrix,RSpectraandmethodsare removed from Imports (now httr, jsonlite, stats, utils).ncomp = 2, ignored arguments.tests/(needs gosset, not a dependency) moved todev/: since Review round 1: tests in R CMD check, API errors, safer rmGeoIdentity #14 R CMD check runs everything intests/.R CMD INSTALLerrors as annotations, to diagnose the load failure seen on Review round 1: tests in R CMD check, API errors, safer rmGeoIdentity #14.https://claude.ai/code/session_01LpE9sTbGPftSMogW6KpkrT
Generated by Claude Code