Test the pythons xgt supports, and bring the README up to date - #9
Conversation
|
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. |
There was a problem hiding this comment.
💡 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".
|
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. |
|
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. |
AI: The 3.9 job fails because xgt no longer runs there:
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:
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.