Skip to content

Test the pythons xgt supports, and bring the README up to date - #9

Merged
landwehrj merged 6 commits into
mainfrom
modernise-python-support
Aug 28, 2026
Merged

landwehrj merged 6 commits into
mainfrom
modernise-python-support

Conversation

@landwehrj

@landwehrj landwehrj commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

AI: The 3.9 job fails because xgt no longer runs there:

ERROR: Package 'xgt' requires a different Python: 3.9.25 not in '<3.15,>=3.10'

Both workflows now test 3.10 through 3.14, which is what xgt asks for. That is the whole of the CI fix.

The README had drifted alongside it:

  • It told people to install python 3.9 to 3.13, in two places. Now 3.10 to 3.14.
  • It pointed at trovares/xgt on Docker Hub. The image is published as rocketgraph/xgt, which is what CI pulls. It also said two versions were available and then listed one, so that is gone too.
  • The GitHub stars badge pointed at trovares/pyxgt rather than this repository.
  • The Twitter badge is dead, so it is removed rather than left pointing at nothing.
  • The note about pip invoking python 2 and needing pip3 has outlived python 2. Replaced with a virtual environment, which is what someone on a modern distribution actually hits when pip refuses to install into a managed python.

One thing left alone and worth a look separately: the CloudFormation quick launch button points at https://trovaresxgt.s3.us-west-2.amazonaws.com/cfxgt.json, which answers 403. AWS/cfxgt.json is in this repository, so the template exists, but the bucket it is served from either moved or is no longer public. I have not guessed at a replacement URL.

@landwehrj

landwehrj commented Aug 28, 2026

Copy link
Copy Markdown
Contributor Author

AI: Pushed two more things.

The CloudFormation quick launch button is gone. Its template answered 403, so the button led nowhere. The other two AWS routes are untouched, and AWS/cfxgt.json is still in the repository for anyone who wants it.

Server setup now leads with the installer at Rocketgraphai/install rather than with AWS, since that is a single command on Linux and macOS and brings its own dependencies. It also links Mission Control, which is what the installer opens when it finishes.

Checked every link in the README afterwards. They all answer 200 apart from anaconda.org, which returns 403 to curl but loads in a browser.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6c4715421f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread README.md
@landwehrj

landwehrj commented Aug 28, 2026

Copy link
Copy Markdown
Contributor Author

AI: Dropped the self run AWS cloud instance. It pointed at AWS/cfxgt.json and AWS/launchxGT.ipynb, neither of which has been touched meaningfully since October 2021, and it was the only thing in the README that needed boto3, so boto3 comes out of the client setup with it. The AWS Marketplace route is untouched.

On how old that is, from PyPI: xgt 2.3.0 was released on 10 June 2025, so about fourteen months ago, and there have been six releases since. The latest is 2.7.0, on 3 August 2026.

The AWS directory is still in the repository. Nothing in the README refers to cfxgt.json, launchxGT.ipynb or launch_utils.py any more, so those three are now unreferenced and could go in a separate change. Marketplace.md is still linked, so it stays.

@landwehrj

landwehrj commented Aug 28, 2026

Copy link
Copy Markdown
Contributor Author

AI: Restructured so installing comes first, and removed the EC2 launcher.

The README opened with about fifty five lines of anaconda, brew and yum before reaching the one command that gets you a running system. The reason given for that order was that client packages were needed to set up some of the servers, which was about boto3 for the EC2 launcher, and that is gone. So it now reads: install Rocketgraph, install the client, demos, then the other ways to run a server for the cases the installer does not cover.

The platform specific python instructions are kept, just condensed to a paragraph rather than three sections, since the substance was install python then pip install xgt.

AWS/cfxgt.json, AWS/launchxGT.ipynb and AWS/launch_utils.py are removed. They only referenced each other, nothing else in the repository or the README pointed at them, and cfxgt.json was last touched in October 2021. AWS/Marketplace.md stays, since the Marketplace route still links it.

README is 117 lines down to 89. Every link answers 200 except anaconda.org, which 403s to curl and loads in a browser.

@landwehrj landwehrj self-assigned this Aug 28, 2026
@landwehrj landwehrj added documentation Improvements or additions to documentation enhancement New feature or request labels Aug 28, 2026
@landwehrj
landwehrj merged commit b844b23 into main Aug 28, 2026
5 checks passed
@landwehrj
landwehrj deleted the modernise-python-support branch August 28, 2026 00:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant