Underscore vs. hyphen in cli arg `mp_20` correct, `mp-20` not

txie-93 / cdvae

An SE(3)-invariant autoencoder for generating the periodic structure of materials [ICLR 2022]

MIT License

211 stars 85 forks source link

Open sgbaird opened 2 years ago

sgbaird commented 2 years ago

Not a big deal, but could add mp-20, perov-20, and carbon-20 in addition to the underscored versions for data=...

sgbaird commented 2 years ago

Similarly, it has a third set of definitions, appearing as mp20, carbon, and perovskite in dict keys for the evaluation metrics.