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

Remove forcefield from data package after it is made installable on other repo #204

Open
davidlmobley opened this issue Feb 9, 2017 · 4 comments

Comments

@davidlmobley
Copy link
Contributor

This PR included a version of the forcefield in the data package which is installed here: #203

A better long-term solution is to make the versioned forcefield itself conda-installable. openforcefield/smirnoff99Frosst#13

@bmanubay
Copy link

bmanubay commented Feb 9, 2017

Hey @davidlmobley,

Just to clarify: there is currently no way to load an ffxml with smarty unless it is in the proper data directory on your os.path, correct? Michael and I were discussing how it would be convenient if the ffxml could be loaded locally, or if the file was conda-installable into the proper directory in smarty.

@davidlmobley
Copy link
Contributor Author

@bmanubay -- right now, what you are saying is correct if you're after loading an ffxml from within the repo. Of course if you provide your own ffxml you can load it. :)

However, the latest master now includes the main smirff99frosst.ffxml in its data directory so it gets installed -- see #203 . As soon as we cut a new release and update the conda recipe (which might or might not be very soon depending on whether this is important to you) it would be conda installable, at least temporarily (cc @nathanmlim ).

See also the other issues cross-linked there, as this is not the best long-term solution.

Is that clear? Sorry, I'm rushing.

@bmanubay
Copy link

Crystal clear David! Having smirff99frosst.ffxml in the latest smarty version will suffice just fine for now, but I do agree that having the ffxml files be conda-installable would be AWESOME from a more collaborative development standpoint :)

@davidlmobley
Copy link
Contributor Author

@bmanubay - see tentative plan here: openforcefield/smirnoff99Frosst#13

If this is something which becomes more urgent for you please comment again.

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

No branches or pull requests

2 participants