Skip to content
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

Mac CI won't execute electric field output #133

Open
lilyminium opened this issue Sep 9, 2024 · 2 comments · May be fixed by #159
Open

Mac CI won't execute electric field output #133

lilyminium opened this issue Sep 9, 2024 · 2 comments · May be fixed by #159

Comments

@lilyminium
Copy link
Collaborator

The cell where the model fails with the error (PR #131):

=================================== FAILURES ===================================
_ examples/train-gnn-to-electric-field/train-gnn-to-electric-field.ipynb::Cell 13 _
[gw2] darwin -- Python 3.11.9 /Users/runner/micromamba/envs/openff-nagl-test/bin/python
Notebook cell execution failed
Cell 13: Timeout of 5 seconds exceeded waiting for output.

Input:
trainer = pl.Trainer(
    max_epochs=100,
    callbacks=[TQDMProgressBar()], # add progress bar
    accelerator="cpu"
)
 

trainer.fit(
    training_model,
    datamodule=training_model.create_data_module(verbose=False),
)

Ubuntu executes fine. It also executes fine on my Mac.

@lilyminium
Copy link
Collaborator Author

AFAICT the options here are:

  • try to work out if I can skip executing that cell. Not optimal since it does execute fine on Ubuntu and is a good test of whether training the model works
  • See if I can just skip that notebook on mac?
  • Work out what a 'Timeout of 5 seconds exceeded waiting for output.' means, since I think that cell takes longer than 5 seconds even on Ubuntu. Is it that it's not writing to stderr/stdout for 5 seconds? If so, should the two lines be split up?
  • Decrease the dataset even more to see if that speeds this cell up -- but this makes the example worse and I'm not sure gains us much

@lilyminium
Copy link
Collaborator Author

Turned off for now. Might be a problem GPU runners can solve.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant