Skip to content

Conversation

gingerknight
Copy link

@gingerknight gingerknight commented Oct 10, 2025

Issue #, if available: n/a

Notes

Implements a new model that can simulate a variety of common errors based on an

Changes

  • New badModel with color-based behavior triggering
    • Server error simulation (red dominant color)
    • Malformed JSON response simulation (green dominant color)
    • RFC7946 schema violation simulation (purple dominant color)
    • Timeout simulation (blue dominant color)
    • Normal detection fallback for other colors

Files Added/Modified

  • src/aws/osml/models/badModel/app.py - Main badModel application with error simulation
  • src/aws/osml/models/badModel/__init__.py
  • test/aws/osml/models/badModel/test_badModel_model.py - Model unit tests

Impact

  • Provides controlled failure simulation for integration testing

Checklist

Before you submit a pull request, please make sure you have the following:

  • Code changes are compact and well-structured to facilitate easy review
  • Changes are documented in the README.md and other relevant documentation pages
  • PR title and description accurately reflect the changes and are detailed enough for historical tracking
  • PR contains tests that cover all new code and the code has been manual tested
  • All new dependencies are declared (if any), and no unnecessary libraries are added
  • Performance impacts (if any) of the changes are evaluated and documented
  • Security implications of the changes (if any) are reviewed and addressed
  • I have read the Contributing Guidelines and agree to follow the Code of Conduct

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@jtblack-aws
Copy link
Contributor

We should update the README as part of this change. Specifically this part

Note: The MODEL_SELECTION environment variable can be used to pick the model to run. Currently, 
we support 3 different types of a model and below are the appropriate naming convention:
- centerpoint
- flood
- aircraft

We don't currently do this, but a short sentence describing the purpose of each would be helpful if we are updating it anyway.

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