Skip to content

Modified dependencies and .yml file#282

Open
ponkrshnan wants to merge 38 commits into
Developmentfrom
feature/azure_pipeline
Open

Modified dependencies and .yml file#282
ponkrshnan wants to merge 38 commits into
Developmentfrom
feature/azure_pipeline

Conversation

@ponkrshnan
Copy link
Copy Markdown
Collaborator

@ponkrshnan ponkrshnan commented May 25, 2026

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 x in all the boxes that apply:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

Go over all the following points, and put an x in all the boxes that apply.
If you're unsure about any of these, don't hesitate to ask. We're here to help!

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

dimtsap and others added 30 commits August 19, 2025 20:18
…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
Updated GitVersion version specification and added comments for clarity.
This is required for GitVersion 6.x
@ponkrshnan ponkrshnan requested a review from dimtsap May 26, 2026 14:16
Comment thread setup.py
'sphinxcontrib_bibtex == 2.5.0',
'Sphinx==6.1.3',
"dev": [
"pytest == 8.2.0",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Are you sure the package version are updated to the latest? pytest latest for example is currently 9.0.3

Comment thread setup.py
package_dir={"": "src"},
package_data={"": ["*.pdf"]},
python_requires='>3.9.0',
python_requires=">3.9.0",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Python 3.9 is at EOL. Let's move to 3.10+ maybe? https://devguide.python.org/versions/

Comment thread pyproject.toml
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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

Comment thread GitVersion.yml
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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?

Comment thread setup.py


# legacy Install
if len(sys.argv) > 2 and sys.argv[2] in ["develop", "install"]:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We should indeed remove this deprecated method. I would suggest migrating to pyproject.toml entirely

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.

3 participants