Modified dependencies and .yml file#282
Conversation
Development
…h itam. still testing stochastic functions using seeds.should change in future to calculate moments of the RVs?
…g disp for minimize optimizer as it is being deprecated, assigning item for montecarlo sampling instead of array
…itical properties of the distribution
Feature/azure pipeline
Updated GitVersion version specification and added comments for clarity.
This is required for GitVersion 6.x
| 'sphinxcontrib_bibtex == 2.5.0', | ||
| 'Sphinx==6.1.3', | ||
| "dev": [ | ||
| "pytest == 8.2.0", |
There was a problem hiding this comment.
Are you sure the package version are updated to the latest? pytest latest for example is currently 9.0.3
| package_dir={"": "src"}, | ||
| package_data={"": ["*.pdf"]}, | ||
| python_requires='>3.9.0', | ||
| python_requires=">3.9.0", |
There was a problem hiding this comment.
Python 3.9 is at EOL. Let's move to 3.10+ maybe? https://devguide.python.org/versions/
There was a problem hiding this comment.
If you use pyproject.toml then deprecate the setup.py. Here is a more complete version of pyproject.toml to extend yours. https://github.com/DTsapetis/UQpy/blob/master/pyproject.toml
There was a problem hiding this comment.
The git version has changed to 6+ and also the internals of this file. This is quite tricky to get right. Have you tested it that it will not break the current versioning and if yes how?
|
|
||
|
|
||
| # legacy Install | ||
| if len(sys.argv) > 2 and sys.argv[2] in ["develop", "install"]: |
There was a problem hiding this comment.
We should indeed remove this deprecated method. I would suggest migrating to pyproject.toml entirely
Modified dependencies and .yml file
Updated dependencies in the yml file for the Azure pipeline. Modified setup.py to install the latest packages
Description
The dependencies in the yml file are updated to reflect the latest package versions in the Azure pipeline. The setup is modified such that the UQpy dependencies, such as Numpy, are at their latest versions.
Related Issue
Resolve #266 #271
Motivation and Context
This change is required for continuous integration in the Azure pipeline and to maintain updated package versions.
How Has This Been Tested?
Tests are performed following UQpy conventions.
Screenshots (if appropriate):
Types of changes
What types of changes does your code introduce? Put an
xin all the boxes that apply:Checklist:
Go over all the following points, and put an
xin all the boxes that apply.If you're unsure about any of these, don't hesitate to ask. We're here to help!