-
Notifications
You must be signed in to change notification settings - Fork 18
Bug report: Application crashed with MissingEntryPointError (WorkGraph) #1209
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
Comments
Ah OK, I realize the issue is actually another! The plugin is not installed. |
I was testing the loading of the muon calculation in case no muon plugin was there, but actually workgraph was installed, that's why I did not catch the issue. However, looking at the code in the qe app, I am not sure we can easily show the other result panels and just put a message in the muon one. The problem seems to be the Would it be fine to just not show any result in case the plugin is not installed, but showing a message like the one you wrote, @giovannipizzi? I think this is much easier to be implemented and, in the end, we need the plugin to be installed. |
You mean that message somewhere in the results panel(s)? Yes, I think that's enough |
yes, precisely |
Sure, we can do this |
I solved this here: #1216. we actually put a check when trying to import an example: |
Hi @mikibonacci , thanks for the quick fix. Add the warning message here, which can indeed let people be aware when importing the examples. |
Hi @superstar54 , I agree with you, we should also handle the issue if a direct upload of a node is done. Let's keep this in mind, because the solution may not be as simple as this. For example, the error printed in this case would be that the workgraph process is not recognized (because aiida-workgraph was not installed), and this was not detected during the muon panel rendering, but when called_descendants was called somewhere in step 4 (you can reproduce the error uninstalling the plugin and loading a muon calc). So, the user will not understand that the plugin needs to be installed. I have no clear solution in mind, right now. Let's discuss when I come back |
Automated report
This issue was created with the app's automated bug reporting feature.
Attached to this issue is the full traceback as well as an environment
fingerprint that contains information about the operating system as well as all
installed libraries.
Additional comments (optional):
I opened the demo server, installed the most recent version of the AiiDAlab app, imported benzene and one of the three LiF muons examples (not sure anymore which of these three, it's the one with UUID
13ca2fc7-96e0-4b4d-b1ca-702ec2694bea
). This error shows trying to open the LiF example. I guess one just needs to install the workgraph? (I confirm that apip install aiida-workgraph
fixes the problem).Attachments
Traceback
Environment fingerprint
By submitting this issue I confirm that I am aware that this information can
potentially be used to determine what kind of calculation was performed at the
time of error.
The text was updated successfully, but these errors were encountered: