You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
With the basic batch create command, player count adjustment isn't necessary, since batch fill ignores any players with no data. However pre-filling data during batch create is very useful for values that apply to the overall event, and this ends up creating dummy players that only contain the pre-filled data. It'd be cleaner if you could simply set the player count when generating the CSV.
The text was updated successfully, but these errors were encountered:
I thought about having something like this as well, but currently gave it lower priority, as I process the CSV files using Excel anyways, and deleting some of the columns therein is only a couple of clicks. However, I agree that other people might do this in a different way and might be glad for such an option.
So yes, I think the proper way is to add an additional optional flag to the batch create command to give GMs the option to specify the number of players while keeping the current number of entries as default.
Implementing an additional flag itself is not much work, but currently I still plan to have some proper overhaul of the whole configuration thingy, which is quite a mess at the moment. I'm planning to have a simple config interface that allows easy access both to the settings of command line flags and also to configuration stored either in the pfscf dir or in the users home dir.
Had a first look at the viper framework two weeks ago, but was not really convinced so far...
With the basic
batch create
command, player count adjustment isn't necessary, sincebatch fill
ignores any players with no data. However pre-filling data duringbatch create
is very useful for values that apply to the overall event, and this ends up creating dummy players that only contain the pre-filled data. It'd be cleaner if you could simply set the player count when generating the CSV.The text was updated successfully, but these errors were encountered: