RMI-PACTA / workflow.pacta

Other
2 stars 3 forks source link

Accept multiple input CSVs #35

Closed AlexAxthelm closed 4 days ago

AlexAxthelm commented 7 months ago

Expected behavior:

AB#11685

AlexAxthelm commented 7 months ago

CC @cjyetman @jdhoffa for opinions

change in config json would god from:

{
 "portfolio_path": "input_dir/default_portfolio.csv",
}

to

{
 "portfolio_path": [
    "input_dir/default_portfolio.csv"
  ],
}

or

{
 "portfolio_path": [
    "input_dir/portfolio_1.csv",
    "input_dir/portfolio_2.csv",
    "input_dir/portfolio_3.csv"
  ],
}
jdhoffa commented 3 weeks ago

@AlexAxthelm I found this issue when reviewing some open discussions with Silicon Ally, is this still an important update?

If yes, I'll make sure to get it on the next sprint :-)