Skip to content

corrected difficulty ordering logic wise and on UI#150

Open
quirky-blu wants to merge 1 commit intoOREL-group:developfrom
quirky-blu:moonshot-earthshot-fix
Open

corrected difficulty ordering logic wise and on UI#150
quirky-blu wants to merge 1 commit intoOREL-group:developfrom
quirky-blu:moonshot-earthshot-fix

Conversation

@quirky-blu
Copy link
Copy Markdown
Contributor

@quirky-blu quirky-blu commented Mar 16, 2026

Summary of changes

Fixes incorrect difficulty ordering for issue categorization. moon_shot and earth_shot were swapped in both the issues_history initialization and the difficulty branching logic (< 2 and < 4 conditions).

The correct order from easiest to hardest is: earth_shot < moon_shot < mars_shot, as originally established in the RL-based ABM work and confirmed by maintainers.

Changes made in run_llamosc_frontend.py:

  • Swapped moon_shot and earth_shot in issues_history dict initialization (lines ~334–335)
  • Fixed difficulty branching so difficulty < 2 maps to earth_shot and difficulty < 4 maps to moon_shot, in both the pending tracking and solved tracking blocks

now the ui also shows the correct difficulty order [top to bottom]
old ui:
{6D5ED9DC-C1A5-48B0-8E16-28027D599A67}
new ui:
{77DE8598-3AFB-451C-B5B6-2DDD8AE5FC93}

Related issue(s)

none

Agent logic / prompts

  • I have not modified any agent logic or prompts.

Tests

  • This PR does not require new tests / no tests added.

Checklist

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.

1 participant