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

Update coloredlogs requirement #939

Merged
merged 5 commits into from
Jul 9, 2024
Merged

Conversation

paulf81
Copy link
Collaborator

@paulf81 paulf81 commented Jul 8, 2024

Update requirements

FLORIS uses the compatible release clause (~=) when specificying versioning (c.f. https://peps.python.org/pep-0440/#compatible-release). This PR updates to newer versions of certain requirements that have had major releases to catch up with: numpy and colored-logs.

It further adds macos to list of OS to try in CI

Related issue

Issue #938

@paulf81 paulf81 added the enhancement An improvement of an existing feature label Jul 8, 2024
@paulf81 paulf81 requested review from rafmudaf and misi9170 July 8, 2024 17:14
@paulf81 paulf81 self-assigned this Jul 8, 2024
@misi9170
Copy link
Collaborator

misi9170 commented Jul 8, 2024

Linking #748

setup.py Outdated
@@ -18,7 +18,7 @@
"attrs",
"pyyaml~=6.0",
"numexpr~=2.0",
"numpy~=1.20",
"numpy~=2.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

Since numpy 2.0 is a very new release (June 16, 2024), it would be helpful for me (and I suspect many other users) to not yet require version 2.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Makes sense, reverting but keeping the 2.0 improvements for a shorter hop later

@misi9170 misi9170 changed the title Update Requirements Update coloredlogs requirement Jul 8, 2024
@paulf81 paulf81 merged commit 1ac4de2 into NREL:develop Jul 9, 2024
8 checks passed
@paulf81 paulf81 deleted the feature/update_req branch July 9, 2024 15:35
@misi9170 misi9170 mentioned this pull request Jul 15, 2024
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An improvement of an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants