-
Notifications
You must be signed in to change notification settings - Fork 14
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
Why remove StructureExampleWidget? #152
Comments
Yes, Nicola did not like it. |
Well, it is a useful feature during developing. |
I agree. What is your opinion on this @giovannipizzi ? |
I also would be in favour of keeping it - it's always better to have something easy to quickly test without thinking/clicking too much. Indeed, Nicola did not like it, I think. Maybe two things:
|
That's probably the right move. 👍
I would prefer to avoid this if possible. |
For debug purposes, I would suggest also including Au for metal. |
Is there any reason why you remove
StructureExampleWidget
from structure_manerger_widget importer?The text was updated successfully, but these errors were encountered: