No caso de demandas não-sazonais, os parâmetros estão sendo usados de forma confusa.
Se o parâmetro sen_peaks for None as demandas são não-sazonais. Mas o que define se a perturbação vai segue distribuição Normal ou não é o parâmetro std. E, portanto, o parâmetro perturb_norm é completamente ignorado.
O ideal seria que o parâmetro perturb_norm fosse usado para definir se as perturbações seguem a distribuição Normal ou não. E, no caso de usar a distribuição uniforme, talvez devesse gerar um erro se std fosse diferente de None.
No caso de demandas não-sazonais, os parâmetros estão sendo usados de forma confusa.
Se o parâmetro
sen_peaksforNoneas demandas são não-sazonais. Mas o que define se a perturbação vai segue distribuição Normal ou não é o parâmetrostd. E, portanto, o parâmetroperturb_normé completamente ignorado.https://github.com/caburu/gym-supplychain/blob/daaa273deacaf72e42eec913138618d28c2db556/gym_supplychain/envs/demands_generator.py#L20-L27
O ideal seria que o parâmetro
perturb_normfosse usado para definir se as perturbações seguem a distribuição Normal ou não. E, no caso de usar a distribuição uniforme, talvez devesse gerar um erro sestdfosse diferente deNone.