Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for loading winiks from disk #29

Closed
ThomasThelen opened this issue Oct 25, 2020 · 0 comments
Closed

Add support for loading winiks from disk #29

ThomasThelen opened this issue Oct 25, 2020 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers
Milestone

Comments

@ThomasThelen
Copy link
Collaborator

ThomasThelen commented Oct 25, 2020

We should add support to the winik_manager manager class for loading pre-defined winiks and populating the manager with them. The file that's being loaded will most likely be a CSV file where each row represents a new winik.

load()

The pseudo-code for this method looks like

initial_winik_file <- read.csv('winiks.csv)
for (line in initial_winik_file) {
   new_winik <- winik$new(name=line$name, father_id=line$father_id, .....)
   self$add(new_winik)

The file will look something like
winik

@ThomasThelen ThomasThelen added enhancement New feature or request good first issue Good for newcomers labels Oct 25, 2020
@ThomasThelen ThomasThelen added this to the 1.0 milestone Oct 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant