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

Fix the elasticsearch, numpy versions #808

Merged

Conversation

athiruma
Copy link
Collaborator

@athiruma athiruma commented Jun 19, 2024

Type of change

Note: Fill x in []

  • bug
  • enhancement
  • documentation
  • dependencies

Description

elastic search client version must satisfy the host version.

python=3.8 :
numpy must be at 1.24.4
So added numpy<=1.26.4 to support for all versions
Solved the following issue:
AttributeError: np.float_was removed in the NumPy 2.0 release. Usenp.float64 instead.

For security reasons, all pull requests need to be approved first before running any automated CI

@athiruma athiruma added bug Something isn't working ok-to-test PR ok to test labels Jun 19, 2024
@athiruma athiruma requested a review from ebattat June 19, 2024 12:10
@athiruma athiruma self-assigned this Jun 19, 2024
requirements.txt Outdated
@@ -19,8 +19,9 @@ google-cloud-bigquery==3.5.0
google-cloud-billing==1.9.1
ibm_platform_services==0.27.0
myst-parser==1.0.0
numpy==1.26.4
Copy link
Collaborator

Choose a reason for hiding this comment

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

@athiruma, failed use 1.24.4 instead

Copy link
Collaborator

@ebattat ebattat Jun 19, 2024

Choose a reason for hiding this comment

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

pls add comment with opensearch 1.2.4 for numpy/pandas/elasticsearch

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done

@athiruma athiruma force-pushed the fix_elastic_search_version branch 3 times, most recently from ab733a5 to a4b0281 Compare June 19, 2024 12:39
Copy link
Collaborator

@ebattat ebattat left a comment

Choose a reason for hiding this comment

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

/LGTM

@athiruma athiruma force-pushed the fix_elastic_search_version branch from a4b0281 to 0ebce4c Compare June 19, 2024 16:01
@athiruma athiruma changed the title Fix the elasticsearch, numpy, pandas versions Fix the elasticsearch, numpy versions Jun 19, 2024
@athiruma athiruma force-pushed the fix_elastic_search_version branch 2 times, most recently from 2f1e3a1 to e0f75c7 Compare June 19, 2024 16:44
@athiruma athiruma force-pushed the fix_elastic_search_version branch from e0f75c7 to 88e28f1 Compare June 19, 2024 16:47
@ebattat ebattat merged commit ac120a8 into redhat-performance:main Jun 19, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ok-to-test PR ok to test
Projects
Development

Successfully merging this pull request may close these issues.

2 participants