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

Python 3.12 #298

Merged
merged 25 commits into from
Jul 2, 2024
Merged

Python 3.12 #298

merged 25 commits into from
Jul 2, 2024

Conversation

dmichaels-harvard
Copy link
Contributor

@dmichaels-harvard dmichaels-harvard commented Jun 27, 2024

  • Updates related to Python 3.12.

    • Had to update venusian (from 1.2.0) to 3.1.0.
    • Had to update pyramid (from 1.10.4) to 1.10.8 (for imp import not found).
      • Had to add pmdarima (no module pyramid.compat).
      • Had to define/update numpy (to 1.26.4) for this as it was implicitly,
        due to something else, using 1.24.4 which failed to build with Python 3.12.
        • And had to update lower bound of Python (from 3.8.1) to 3.9 for this.
    • Had to dcicutils (from 8.11.0) to 8.13.0 (for pyramid update for imp import not found).
  • Minor change to dev_servers.py to facilitate running a local ElasticSearch proxy
    to observe traffic (resquests/responses) between the portal and ElasticSearch
    with a tool like mitmproxy or mitmweb; see comments in dev_server.py.

@@ -0,0 +1,24 @@
# Not that pyramid.compat disappeared with pyramid 2.0.2, during move to Python 3.12,
Copy link
Member

Choose a reason for hiding this comment

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

Note*

# mitmweb; e.g. setting elasticsearch.server.actual_port to 9201 and elasticsearch.server to localhost:9200
# will case ElasticSearch to actually run on port 9201 but will cause portal to talk to it via port 9200,
# and then we can run mitmweb --mode reverse:http://localhost:9201 -p 9200 --web-port 8081 which will
# allow us to browse to http://localhost:8081 locally to observe all of the ElasticSearch traffic.
Copy link
Member

Choose a reason for hiding this comment

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

Would be interested to hear how you're using this currently

@dmichaels-harvard dmichaels-harvard merged commit 02cac24 into master Jul 2, 2024
7 checks passed
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.

2 participants