log as input argument not necessary?

[Alireza Khorshidi]

It seems that we don't need to feed in "log" as input argument. It can already be accessed by "self.parent.log"

[andrew_peterson]

Could you provide a link?

If I remember right, the reason I included this specifically was so you could access things like descriptors or models without having to link them to a calculator. E.g., if you just want to make some fingerprints.

For example, see the script here. A logger could be provided to "Gaussian" if desired. But it has no parent, so it would crash if trying to get the logger from the parent.

[Alireza Khorshidi]

Sorry, forgot to put the link. Here:

https://bitbucket.org/andrewpeterson/amp/src/363187c54be5bc99f027bd89604911ed85574509/model/neuralnetwork.py?at=master&fileviewer=file-view-default#neuralnetwork.py-112

[Alireza Khorshidi]

It makes sense to have log as input argument to have the method stand-alone.