-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Is MolGAN incompatible with max_atoms=31 or ZINC250k dataset #3962
Comments
CC @shreyasvinaya who may have some insights |
Hi @UmarZein |
I haven't been able to get a good run
when running
^after that line, it eats 18GB of VRAM (dunno if this is it's actual size or whether there is something wrong with the docker container) |
@shreyasvinaya Is the new torch molgan stable to run? |
❓ Questions & Help
I have replicated 90% of https://deepchem.io/tutorials/generating-molecules-with-molgan/
I assume it's more of a problem on the model/architecture rather than the library's implementation, but if anyone has used this library's MolGAN to fit an at least equally complex dataset please say so
with the differences being:
num_atoms=31
(95% of the molecules in the dataset has at most 31 atoms)atom_labels=[0, 6, 7, 8, 9, 15, 16, 17, 35, 53]
MolGAN(learning_rate=ExponentialDecay(0.0001, 0.9, 5000), vertices=max_atoms, model_dir="model_dir")
MolGAN(learning_rate=ExponentialDecay(0.001, 0.9, 5000), vertices=max_atoms, model_dir="model_dir")
gan.fit_gan(iterbatches(25), generator_steps=0.2, checkpoint_interval=1000, max_checkpoints_to_keep=25, restore=False)
dataset.X.shape
is(235043, 31, 31)
dataset.y.shape
is(235043, 31)
because when i fit the model to the data, the generator loss keeps growing:
The text was updated successfully, but these errors were encountered: