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

Dead kernels #67

Open
1 task
bjorgve opened this issue Oct 12, 2021 · 5 comments
Open
1 task

Dead kernels #67

bjorgve opened this issue Oct 12, 2021 · 5 comments

Comments

@bjorgve
Copy link
Member

bjorgve commented Oct 12, 2021

When mrcpp hits a MSG_ABORT. It kills the notebook kernel. We should make sure we find all of these and deal with them
from the vampyr side of the code.

I suggest we list code that lead to a dead kernel here, so that it's easier for us til fix them in the future:

  • fetchEndNode when index is out of bound
@stigrj
Copy link
Contributor

stigrj commented Oct 12, 2021

Do we know how to handle them on the Python side?

@bjorgve
Copy link
Member Author

bjorgve commented Oct 12, 2021

No, but we need to hack it some way. I believe we make sure we never have empty FunctionTrees on the Python side. Am I correct?

Or is this "fixed" in mrcpp?

@robertodr
Copy link
Contributor

robertodr commented Oct 12, 2021 via email

@robertodr
Copy link
Contributor

Using std::optional is another option: https://en.cppreference.com/w/cpp/utility/optional But it's MRCPP-side, not VAMPyR-side.

@bjorgve
Copy link
Member Author

bjorgve commented Oct 13, 2021

I was thinking something like this

 .def("fetchEndNode", [](MWTree<D> &tree, int i) {
    if (does i make sense?) return tree.getMWEndNode(i)

    return pybind11::none;
})

I haven't tested if this actually will work. I'll look at it when I have time

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

3 participants