amazon-science / earth-forecasting-transformer

Official implementation of Earthformer
Apache License 2.0
349 stars 58 forks source link

Default values for CuboidTransformerModel yield assertion error #42

Closed nilsleh closed 1 year ago

nilsleh commented 1 year ago

The default arguments for the CuboidTransformerModel include enc_depth=[4, 4, 4] and dec_depth=[2, 2] and during the init the line assert len(enc_depth) == len(dec_depth) yields an error.

gaozhihan commented 1 year ago

Yes. The default configs specified in the implementation of CuboidTransformerModel are arbitrary. To reproduce the results shown in the paper, you may want to refer to the cfg.yaml file of each dataset for the exact hyperparameter configs (E.g., we set enc_depth = [4, 4] and dec_depth = [4, 4] by default for experiments on N-body MNIST using the corresponding cfg.yaml.).

gaozhihan commented 1 year ago

Let me close the issue for now. Please feel free to reopen if you find any problems.