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

bump AWB to released 2.0 #394

Merged
merged 3 commits into from
May 9, 2023
Merged

bump AWB to released 2.0 #394

merged 3 commits into from
May 9, 2023

Conversation

unkcpz
Copy link
Member

@unkcpz unkcpz commented May 8, 2023

Jinja2 and importlib are from aiida-quantumespresso, therefore the version is loosely pinned. The pybtex has to be added to work around the installation issue.

@unkcpz unkcpz requested a review from superstar54 May 8, 2023 15:26
@unkcpz unkcpz force-pushed the bump-awb-2.0 branch 4 times, most recently from d492323 to f1e9bef Compare May 8, 2023 17:10
widget-bandsplot~=0.5.1
pybtex==0.24.0
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This somehow has to be pinned, because of the issue https://github.com/aiidalab/aiidalab-qe/actions/runs/4917462707/jobs/8782576404. It also fails the test of aiidalab-docker-stack in aiidalab/aiidalab-docker-stack#374

@@ -22,13 +22,12 @@ project_urls =
[options]
packages = find:
install_requires =
Jinja2~=3.0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please keep jinjia2 and importlib-resources here. Otherwise, the dependency is not clear.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, make sense. I delete it for test. You are correct, since it is used in the package, better to keep in the dependencies list.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But I loose the version a bit so it more flexible to the change from upstream packages, e.g aiida-core and aiida-quantumespresso.

@unkcpz unkcpz requested a review from superstar54 May 9, 2023 07:52
@unkcpz unkcpz merged commit e0217c5 into master May 9, 2023
4 checks passed
@unkcpz unkcpz deleted the bump-awb-2.0 branch May 9, 2023 09:06
unkcpz added a commit that referenced this pull request May 9, 2023
`aiidalab-widgets-base` 2.0 is released, the version is bump here. Jinja2 and importlib are from aiida-quantumespresso, therefore the version is loosely pinned. The pybtex has to be added to work around the installation issue.
unkcpz added a commit that referenced this pull request May 9, 2023
`aiidalab-widgets-base` 2.0 is released, the version is bump here. Jinja2 and importlib are from aiida-quantumespresso, therefore the version is loosely pinned. The pybtex has to be added to work around the installation issue.
unkcpz added a commit that referenced this pull request May 9, 2023
`aiidalab-widgets-base` 2.0 is released, the version is bump here. Jinja2 and importlib are from aiida-quantumespresso, therefore the version is loosely pinned. The pybtex has to be added to work around the installation issue.
unkcpz added a commit that referenced this pull request May 9, 2023
`aiidalab-widgets-base` 2.0 is released, the version is bump here. Jinja2 and importlib are from aiida-quantumespresso, therefore the version is loosely pinned. The pybtex has to be added to work around the installation issue.
unkcpz added a commit that referenced this pull request May 17, 2023
workaround

r-andres

bump AWB to released 2.0 (#394)

`aiidalab-widgets-base` 2.0 is released, the version is bump here. Jinja2 and importlib are from aiida-quantumespresso, therefore the version is loosely pinned. The pybtex has to be added to work around the installation issue.

Move module imports from __init__.py of package to app modules (#393)

Remove report legacy support

pinning qe plugin so test pass
unkcpz added a commit that referenced this pull request May 17, 2023
workaround

r-andres

bump AWB to released 2.0 (#394)

`aiidalab-widgets-base` 2.0 is released, the version is bump here. Jinja2 and importlib are from aiida-quantumespresso, therefore the version is loosely pinned. The pybtex has to be added to work around the installation issue.

Move module imports from __init__.py of package to app modules (#393)

Remove report legacy support

pinning qe plugin so test pass
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

Successfully merging this pull request may close these issues.

None yet

2 participants