Skip to content

Conversation

@iamh2o
Copy link

@iamh2o iamh2o commented Oct 18, 2025

Summary

  • add backend support for retrieving analysis worksets from the head node via SSM
  • expose a new manager route and frontend model helper for analysis worksets
  • add an Analysis Worksets tab with localized strings to display workset counts in the cluster view

Testing

  • npm run lint

https://chatgpt.com/codex/tasks/task_e_68f3d6b690388331b12ad25e8248c001

Copy link

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

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.

ℹ️ 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 on lines 48 to +50
from api.validation import validated, EC2Action
from api.validation.schemas import CreateUser, DeleteUser, GetClusterConfig, GetCustomImageConfig, GetAwsConfig, GetInstanceTypes,\
Login, PushLog, PriceEstimate, GetDcvSession, QueueStatus, ScontrolJob, CancelJob, Sacct
from api.validation.schemas import (
CreateUser,

Choose a reason for hiding this comment

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

P0 Badge Fix unexpected indent in app imports

The new AnalysisWorksets schema import was added with leading spaces so it is indented beneath the preceding statement. Python treats this as an IndentationError at module load (python -m py_compile app.py fails with “unexpected indent (app.py, line 49)”), preventing the Flask app from starting and making the whole API unusable. Remove the indentation so the import sits at the top level like the other imports.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants