Skip to content

add psycopg2, PyYAML, and Cartopy dependencies to QGIS 3.28.1#17257

Merged
verdurin merged 2 commits intoeasybuilders:developfrom
branfosj:20230204110210_new_pr_QGIS3281
Feb 13, 2023
Merged

add psycopg2, PyYAML, and Cartopy dependencies to QGIS 3.28.1#17257
verdurin merged 2 commits intoeasybuilders:developfrom
branfosj:20230204110210_new_pr_QGIS3281

Conversation

@branfosj
Copy link
Copy Markdown
Member

@branfosj branfosj commented Feb 4, 2023 •
edited by jfgrimm
Loading

(created using eb --new-pr)

I got a bunch of Couldn't load plugin ... messages on starting QGIS:

ModuleNotFoundError: No module named 'owslib'
ModuleNotFoundError: No module named 'yaml'
ModuleNotFoundError: No module named 'psycopg2'

I'm using Cartopy to supply owslib as QGIS+Cartopy is a sensible combination of modules that would be wanted together - avoiding the potential to have a different version of owslib available from a separate module just providing this.

@branfosj branfosj added this to the next release (4.7.1?) milestone Feb 4, 2023
@boegelbot

This comment was marked as outdated.

Sign in to view
@jfgrimm
Copy link
Copy Markdown
Member

jfgrimm commented Feb 6, 2023

Test report by @jfgrimm
SUCCESS
Build succeeded for 5 out of 5 (1 easyconfigs in total)
himem06.pri.viking.alces.network - Linux CentOS Linux 7.9.2009, x86_64, Intel(R) Xeon(R) Platinum 8160 CPU @ 2.10GHz (skylake_avx512), Python 3.6.8
See https://gist.github.com/42d0731f82d340805fbf495a6b9b969e for a full test report.

@jfgrimm
Copy link
Copy Markdown
Member

jfgrimm commented Feb 7, 2023

@boegelbot please test @ generoso

@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented Feb 7, 2023

@jfgrimm: Request for testing this PR well received on login1

PR test command 'EB_PR=17257 EB_ARGS= EB_CONTAINER= /opt/software/slurm/bin/sbatch --job-name test_PR_17257 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 10121

Test results coming soon (I hope)...

Details

- notification for comment with ID 1420553366 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented Feb 7, 2023

Test report by @boegelbot
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
cns1 - Linux Rocky Linux 8.5, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/0b21a4d4350508d1635c5aef26fcdaad for a full test report.

@branfosj
Copy link
Copy Markdown
Member Author

branfosj commented Feb 7, 2023

Test report by @branfosj
SUCCESS
Build succeeded for 64 out of 64 (1 easyconfigs in total)
bear-pg0105u03a.bear.cluster - Linux RHEL 8.6, x86_64, Intel(R) Xeon(R) Platinum 8360Y CPU @ 2.40GHz (icelake), Python 3.6.8
See https://gist.github.com/95b5f257d2d0b38a6c983ecd00d06ad4 for a full test report.

@verdurin
Copy link
Copy Markdown
Member

verdurin commented Feb 13, 2023

Test report by @verdurin
SUCCESS
Build succeeded for 55 out of 55 (1 easyconfigs in total)
easybuild-c7.novalocal - Linux CentOS Linux 7.9.2009, x86_64, Intel Xeon Processor (Skylake, IBRS), Python 3.6.8
See https://gist.github.com/2bb463c49d2d500c8f9ad4aea9720137 for a full test report.

verdurin
verdurin approved these changes Feb 13, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@verdurin verdurin left a comment

Choose a reason for hiding this comment

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

Looks fine.

@verdurin
Copy link
Copy Markdown
Member

verdurin commented Feb 13, 2023

Going in, thanks @branfosj!

@verdurin verdurin merged commit 5e1596c into easybuilders:develop Feb 13, 2023
@branfosj branfosj deleted the 20230204110210_new_pr_QGIS3281 branch February 13, 2023 14:04
@boegel boegel changed the title add psycopg2, PyYAML, and Cartopy deps to QGIS add psycopg2, PyYAML, and Cartopy dependencies to QGIS 3.28.1 Feb 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@verdurin verdurin verdurin approved these changes

Assignees

No one assigned

Labels

bug fix

Projects

None yet

Milestone

4.7.1

Development

Successfully merging this pull request may close these issues.

4 participants

@branfosj @boegelbot @jfgrimm @verdurin