Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

RTR Robustness Updates #205

Merged
merged 7 commits into from
Oct 17, 2024
Merged

RTR Robustness Updates #205

merged 7 commits into from
Oct 17, 2024

Conversation

ChristopherHammond13
Copy link
Member

RTR Robustness

  • Enhancement
  • Major feature update
  • Bug fixes
  • Breaking change
  • Updated unit tests
  • Documentation

Issues resolved

  • Should resolve [ BUG ] Successful RTR connections when they fail #187
  • Resolves some various RTR logging issues
  • Resolves some potential issues arising from a potentially incorrect usage of executor.map() for large RTR sessions
  • Bumps dependencies across the board
  • Bumps version to 0.8.1

@ChristopherHammond13 ChristopherHammond13 added bug 🐛 Something isn't working enhancement 🌟 New feature or request labels Sep 11, 2024
@ChristopherHammond13 ChristopherHammond13 self-assigned this Sep 11, 2024
@ChristopherHammond13
Caracara 0.9.0: Preparation for Python 3.13 (and huge code cleanup) (#…
5bc830c 
…206)

* Major code formatting cleanup, deprecation of Python 3.8, and removal of setuptools requirement

* Add note about Python version compatibility

* Temporarily set Python 3.13 CI version to 3.13 RC2

* Changes to sate isort and black together

* Unified development dependencies

* Bumps py7zr to 0.22.0 as we no longer support Python 3.7. Also, bumps caracara-filters to 1.0.0+.

* Bumps dependencies
@ChristopherHammond13
Resolves #187 and some other general RTR snags. Also bumps dependencies.
cb037ee
@ChristopherHammond13
Resolves linting issues
8f067de
@ChristopherHammond13
Merge branch 'bugfix/187/rtr-connection-failures' of github.com:/Crow…
b8646d5 
…dStrike/caracara into bugfix/187/rtr-connection-failures
@ChristopherHammond13
Resolves linting issues after merge
016047e
@ChristopherHammond13
Copy link
Member Author

I have tested this code as working locally by overwriting the MAX_BATCH_SESSIONS_HOSTS constant to 1, and can see multiple parallel threads working well. Going to merge this.

@ChristopherHammond13 ChristopherHammond13 merged commit e3b3be6 into main Oct 17, 2024
8 checks passed
@ChristopherHammond13 ChristopherHammond13 deleted the bugfix/187/rtr-connection-failures branch October 17, 2024 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jshcodes jshcodes Awaiting requested review from jshcodes

@kenoel kenoel Awaiting requested review from kenoel

Assignees

@ChristopherHammond13 ChristopherHammond13

Labels
bug 🐛 Something isn't working enhancement 🌟 New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[ BUG ] Successful RTR connections when they fail
1 participant
@ChristopherHammond13