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

QueryProvider widget not displaying #437

Open
CasperWA opened this issue Jun 9, 2022 · 8 comments
Open

QueryProvider widget not displaying #437

CasperWA opened this issue Jun 9, 2022 · 8 comments
Labels
bug Something isn't working

Comments

@CasperWA
Copy link
Owner

CasperWA commented Jun 9, 2022

The widget for choosing the provider and database to query is not displayed.

@unkcpz
Copy link
Collaborator

unkcpz commented Jun 9, 2022

The issue comes from the version of ipywidgets-extended. Downgrade it to 1.1.1 from 1.2.0 can solve the issue.
As mentioned #433, I think this package is required by the ExtenedGroup, I will bring this up to Nicola this increase our burden to maintain a widget that is only used by us and to me, it is not necessary for optimadeclient.

EDIT: Ah! that is your package where ExtendedGroup is implemented. Then it must be some subtle dependencies that break this.

@unkcpz
Copy link
Collaborator

unkcpz commented Jun 9, 2022

So for now, we can pinning the version of ipywidget-extended. I open the PR #438.

@unkcpz
Copy link
Collaborator

unkcpz commented Jun 23, 2022

Hi @CasperWA, the https://optimadeclient.matcloud.xyz/ still does not have provider widget working. But it is sure update with the latest one correct? Could you tell me how you push to the dev server, you do it manually from your computer?

@CasperWA
Copy link
Owner Author

The version was not properly avoided in the latest release. I have just released a new version and will update the MaterialsCloud tool accordingly.

@unkcpz
Copy link
Collaborator

unkcpz commented Jun 23, 2022

Good thanks! @CasperWA

I have a meeting with Nicola, he now suggests that we can only keep MC3D, MC2D, and 'CURATED covalent organic' on the list. So can get rid of the ipywidget-extent. What do you think?

@CasperWA
Copy link
Owner Author

I don't want to get rid of the widget unless it's downright disruptive. The list and groupings can be changed in the other dedicated repository in the notebook where the individual widget parts are instantiated.

@unkcpz
Copy link
Collaborator

unkcpz commented Jun 23, 2022

No problem, I think this is also useful in other places. I just need some time to learn how it is organized and can contribute to fixing things quickly.

@CasperWA
Copy link
Owner Author

Will have to pin it to v1.1.1 again, according to #457.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants