Should Model directly subtype AbstractSystem?
#1007
Labels
discussion
Discussion thread of broader scope
Comments
|
Just had a look at the interface, and it seems OK, with the caveat that model is not mutable (and that there is no notion of velocity) |
|
I did this originally when getting atomsbase support up, but @antoine-levitt vetoed it at the time 😄, so no blockers from me. |
|
Did I really? I guess I didn't want to commit to atomsbase at the time (model can only subtype once), but now it's relatively established. I still don't really see what it buys either us or downstream consumers (Model is just a container and has no specific computational cost, so conversions are just copies of pointers, and if you're worried about that cost you probably shouldn't do DFT :-)), but if there's demand why not... |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I'm going through some things to prep my AtomsBase introductory materials for the MolSSI workshop and noticed that at the moment, DFTK provides functions to convert to prototype interface implementations, but why not just (also, or instead) have
Modeldirectly implement the interface and subtypeAbstractSystem? I could see some advantages down the line where if you had to pass information back and forth quite often, you wouldn't want to repeatedly incur the computational cost of conversion.It's certainly not a blocker for any particular thing I'm doing right now, but figured I'd start the conversation as it doesn't seem it would be too difficult to do. I'd be happy to draft a PR if this makes sense!
The text was updated successfully, but these errors were encountered: