Skip to content

Allow for a variable number of tossups per packet#2

Open
Captain-Quack wants to merge 1 commit into
InfiniteStryker0:mainfrom
Captain-Quack:main
Open

Allow for a variable number of tossups per packet#2
Captain-Quack wants to merge 1 commit into
InfiniteStryker0:mainfrom
Captain-Quack:main

Conversation

@Captain-Quack

Copy link
Copy Markdown

Decisions made related to improving the app:

  • changed
     used_subcategories <- list(
         History = c(),
         Science = c(),
         Literature = c(),
         FineArtsOther = c(),
         RMPSS = c()
       )
    
    to
       used_subcategories <- lapply(categories, function(x) character())
    
    because it looks nice
  • To avoid complicated data partitioning, the users input is automatically rounded down to the nearest multiple of 4

Decisions made related to not knowing enough R:

  • deck refreshes when it runs out of samples
  • generate_packet_TUandB just shuffles all bonuses instead of the related quarter (see inner comment)
  • To avoid complicated data partitioning, the users input is automatically rounded down to the nearest multiple of 4

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.

1 participant