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

Fix tutorial04 API #485

Closed
wants to merge 6 commits into from
Closed

Conversation

wenyan4work
Copy link
Contributor

Description

update api of tutorial04, partially resolves #480
forcebalance 1.9.4 doesn't work properly with evaluator 0.4.1, so this fix sticks to 1.9.3+0.3.9

Todos

None

Questions

None

Status

  • Ready to go

@codecov
Copy link

codecov bot commented Jan 19, 2023

Codecov Report

Merging #485 (8c5adbf) into main (66adc50) will decrease coverage by 2.40%.
The diff coverage is n/a.

Additional details and impacted files

@mattwthompson
Copy link
Member

I will try to finally take a look at this next week. Could you confirm my understanding that this relies on running simulations, i.e. should be run on a cluster or workstation with access to GPU compute?

@mattwthompson
Copy link
Member

Curious - does 1.9.5 make this all work again?

@wenyan4work
Copy link
Contributor Author

I will try to finally take a look at this next week. Could you confirm my understanding that this relies on running simulations, i.e. should be run on a cluster or workstation with access to GPU compute?

It runs on my windows 11 laptop running cuda inside WSL-2. I believe it should run on any standard linux + cuda workstation.

I didn't try the DaskJobQueueBackend because I have no access to that type of resources.

The latest commits has been validated with a fresh install of openff-toolkit 0.12 + evalutor 0.4.2 + forcebalance 1.9.5

@wenyan4work
Copy link
Contributor Author

Curious - does 1.9.5 make this all work again?

Yes, it works now... I was editing my comment and your comment popped up at exactly the same moment :)

@mattwthompson
Copy link
Member

Excellent, hope to have some time this week to once this over and think about how it should be deployed. Not sure if a new patch release is in order.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@mattwthompson mattwthompson mentioned this pull request Feb 13, 2023
4 tasks
@mattwthompson
Copy link
Member

@wenyan4work were you able to get the simulations in the latter part of tutorial 2 working? I won't ask you to run them to completion but I'm curious if they were able to start without error and if it seemed like they were actually using your laptop's hardware (especially if you have a discrete GPU)

@wenyan4work
Copy link
Contributor Author

wenyan4work commented Feb 13, 2023 via email

@mattwthompson
Copy link
Member

Could you upload the JSON files that are written out at the end of each of the tutorials?

@mattwthompson
Copy link
Member

Okay - I finally got CUDA up and running on my Windows box so they're churning along. If you have the files handy and feel like sharing I'd still like to use them to double-check mine, but I'm not asking you to re-run the simulations on my account.

@mattwthompson
Copy link
Member

Thanks for your contribution! It is included in #499 with some other housekeeping.

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

Successfully merging this pull request may close these issues.

tutorial data and script are outdated
2 participants