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

v1.2.1 release candidate #97

Open
wants to merge 129 commits into
base: master
Choose a base branch
from
Open

v1.2.1 release candidate #97

wants to merge 129 commits into from

Conversation

ceftanveer
Copy link
Collaborator

@ceftanveer ceftanveer commented Jan 4, 2025

Checklist:

  1. pull changes from master into dev branch (should just be small changes to .md files)

  2. update the documentation if necessary.

  3. ensure the documentation builds are passing on ReadTheDocs

  4. update the QGIS and CanFlood version tag on build.ui and the project README.md.

  5. update the Updates section of the project README.md.

  6. similarly update plugin metadata file

  7. update the plugin requirements file to capture the exact dependencies (pip freeze)

  8. execute all pytests. investigate warnings. fix errors. (ctrl+f in console for 'error' and 'warning' as a workaround)

  9. use plug_zip.bat to create the plugin zip (canflood.zip) installable version of the release candidate in ./plugin_zips

  10. perform a person test by having a non-developer follow relevant tutorials. investigate warnings and fix errors.

cefect and others added 11 commits January 8, 2025 18:47
* notes and catches

* Added Lambda function for explict int typecasting

* Removed debug code and comments

* add some comments and remove a prints tatement

* Code cleanup and added QmessageLog

* fix the underscore import

---------

Co-authored-by: Seth Bryant <[email protected]>
Co-authored-by: Tanveer singh Bhatia <[email protected]>
* add pytest-qt to requirements for proper error catching


---------

Co-authored-by: Tanveer singh Bhatia <[email protected]>
Co-authored-by: Seth Bryant <[email protected]>
clean up some error catches on teh validation
add openpyxl as the explicate engine

---------

Co-authored-by: Tanveer singh Bhatia <[email protected]>
Co-authored-by: Seth Bryant <[email protected]>
Copy link
Collaborator

@cefect cefect left a comment

Choose a reason for hiding this comment

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

Need to close out all the errors first

ceftanveer and others added 18 commits January 21, 2025 21:33
* Fix for Box plot not coming up

* Updated plots for model and build



---------

Co-authored-by: Tanveer singh Bhatia <[email protected]>
closes issue 
* Updated control files with relative path, removed hardcoded path

* Added logic to handle relative path


---------

Co-authored-by: Tanveer singh Bhatia <[email protected]>
Co-authored-by: Seth Bryant <[email protected]>
absolute_fp improper equals
remove not implemented check
pass absolute_fp
clean up docstrings
…ta-type

55 switch tutorial vector layer data type
* add test_build_rsamp_runPrep

* Update 06_tutorials.rst
* absolute_fp checked while updating control file

* updated control file for relative path

* Removed unsed argument

* Updated test to check relative paths

* Added a test folder for second case (False) absolute_fp

* update some comments

* comments

* Update test_tutorial_02.py

* Updated Test and dmg file

* Update CanFlood_control_01.txt

spelling

* comment and logging updates

* clean up entry point and rename

* add version tag to build ui

* move definitions file

* add test_sensi

* test rfda

* Create test_dikes.py

* pythonpath import fix

* fix version tag

* Create CanFlood - tutorial 2.qgz

* plugin setup working directory to cf_fp

---------

Co-authored-by: Tanveer singh Bhatia <[email protected]>
Co-authored-by: Seth Bryant <[email protected]>
* Web connections proof of concept

* Updated with relative paths instead of hardcoded path

* Added arcgis, wms and wcs connections

* replaced print statement with proper error

* Added test for Web connections

* Removed unused ini file path

* Updated test to ensure ini files updated to pass test

* Update test_webconnections.py

rename

* Refactored test code

* Update test_webconnections.py

* update gitignore

* Update webConnections.py

* Removed the non working links

---------

Co-authored-by: Tanveer singh Bhatia <[email protected]>
Co-authored-by: Seth Bryant <[email protected]>
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.

7 participants