Skip to content

Conversation

@devin-ai-integration
Copy link
Contributor

NTN Support for Hub Module

This PR adds Non-Terrestrial Network (NTN) support to the hub module, with improved parameter handling and documentation.

Changes

  • Added out and in_ parameters to hub.sync for selective sync control
  • Updated allow parameter documentation to accurately reflect penalty box behavior
  • Removed undocumented ntn parameter
  • Added comprehensive test coverage for NTN-related functionality
  • Implemented proper parameter name mapping in tests (in_ -> in)

Testing

  • Added test cases for all new parameters and combinations
  • Verified parameter mapping functionality
  • All tests passing across Python versions 3.6 to 3.11

Documentation

  • Updated parameter descriptions to match official Blues documentation
  • Clarified penalty box behavior in allow parameter
  • Added documentation for selective sync parameters

Link to Devin run: https://app.devin.ai/sessions/95ffd763d71b445198b9370e49dcd93f

Co-Authored-By: rlauer@blues.com <rlauer@blues.com>
@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
  • Look at CI failures and help fix them

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Co-Authored-By: rlauer@blues.com <rlauer@blues.com>
@rdlauer rdlauer closed this Feb 10, 2025
@rdlauer rdlauer deleted the devin/1738177804-card-power branch February 10, 2025 16:56
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