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

Issue 133 and 134 merge branch #149

Closed
wants to merge 63 commits into from
Closed

Issue 133 and 134 merge branch #149

wants to merge 63 commits into from

Conversation

HayleyO
Copy link
Contributor

@HayleyO HayleyO commented Aug 8, 2024

Pull request for Issue #133 and Issue #134

HayleyO and others added 30 commits June 20, 2024 13:49
@HayleyO
Confirmed you can add redis configurations to meta data file parsing …
d8d1528 
…with no breaking to local redis hosting defaults
@HayleyO
Add unit test
bbbb0eb
@HayleyO
Add password to unit tests
d7e7d1a
@HayleyO
Removed argument name
0973694
@HayleyO
Updated config to have a neater dictionary of redis server configs + …
35237b3 
…handle those configs
@HayleyO
Updated unit test
d4b5fed
@rachaelchertok1
Modified Redis adapter to work with multiple servers
a5f4eaa
@rachaelchertok1
Adding multi-server experiment
bc28576
@rachaelchertok1
Merge pull request #2 from HayleyO/Issue_134_Create_Multiple_Redis_Se…
f03b346 
…rver_Adapter

Issue 134 create multiple redis server adapter
@HayleyO
Allow access to OnAir from different directories
93b3533
@rachaelchertok1
Updates to redis_adapter unit test and to redis_adapter.py to have su…
e6d8abb 
…bscriptions only be set up if server ping was successful.
@rachaelchertok1
All unit tests passing and 100% code coverage!
259dc53
@rachaelchertok1
Removed commented functions from test redis adapter code and removed …
95c3a47 
…debug prints from redis_adapter.py
@HayleyO
Allow access to OnAir from different directories
dc58c1b
@rachaelchertok1 @HayleyO
Updates to redis_adapter unit test and to redis_adapter.py to have su…
1e82748 
…bscriptions only be set up if server ping was successful.
@rachaelchertok1 @HayleyO
All unit tests passing and 100% code coverage!
1f18f50
@rachaelchertok1 @HayleyO
Removed commented functions from test redis adapter code and removed …
d377e0a 
…debug prints from redis_adapter.py
@HayleyO
Merge branch 'main' of https://github.com/nasa/OnAIR into new_new_branch
f146a9c
@HayleyO
Merge branch 'unit_testing_134_create_multiple_redis_server_adapter' of
119ff9f 
https://github.com/HayleyO/OnAir_SummerInterns2024 into new_new_branch
@rachaelchertok1 @HayleyO
Updates to redis_adapter unit test and to redis_adapter.py to have su…
2cad053 
…bscriptions only be set up if server ping was successful.
@rachaelchertok1 @HayleyO
All unit tests passing and 100% code coverage!
25a1637
@rachaelchertok1 @HayleyO
Removed commented functions from test redis adapter code and removed …
9b17eff 
…debug prints from redis_adapter.py
@dragonejt @HayleyO
Update config file specification format
73cc778
@dragonejt @HayleyO
enable OPTIONS section to be optional
0eeb92e
@dragonejt @HayleyO
fix: Remove dev_enabled and viz_enabled, use GitHub default python gi…
c2350d0 
…tignore, remove multiline f-string
@asgibson @HayleyO
example.ini was incorrect, changed to redis_example.ini
22a51a9
@dragonejt @HayleyO
fix: Update redis_example.ini and kalman_csv_output_example.ini to re…
abf4149 
…flect changes
@dragonejt @HayleyO
fix: revert ExecutionEngine instantiation in tests to use __new__
186de9c
@dragonejt @HayleyO
fix: re-add .gitignore to ignore macOS .DS_Store files
eba10a8
@the-other-james @HayleyO
Add note about github username to contributors section.
d0f7d14
the-other-james and others added 29 commits August 9, 2024 09:04
@the-other-james @HayleyO
Store the message ID lookup table in the telemetry metadata
0fa6703
@the-other-james @HayleyO
Deal with nested structs for cFS messages
98f6c00
@the-other-james @HayleyO
Remove prints
b6f0342
@williamzhang0306 @HayleyO
bug fix for nested message header fields
46369fb
@williamzhang0306 @HayleyO
re working unit tests for sbn_adapter
8c053b2
@williamzhang0306 @HayleyO
fixed str() overriding of mock object
a11909e
@williamzhang0306 @HayleyO
slight change to sbn_adapter get_next to be similar to redis_adapter
e143678
@williamzhang0306 @HayleyO
parse_meta_data_file raises ConfigKeyError if missing 'channels' in json
208ee1c
@williamzhang0306 @HayleyO
potential bugfix for getting fields of nested structs
1eb5fde
@williamzhang0306 @HayleyO
80% testing coverage
f5ffa86
@williamzhang0306 @HayleyO
connect() and message_listener_thread() tests
41c477a
@williamzhang0306 @HayleyO
additional test for get_current_data
7855e72
@williamzhang0306 @HayleyO
Formatting, removed todo comments
c426c7f
@williamzhang0306 @HayleyO
Removed time field and associated unittest from sbn adapter
c91cd7f
@williamzhang0306 @HayleyO
documetation for running with onair draft 1
9fecbb7
@williamzhang0306 @HayleyO
Minor doc and formatting updates.
a3557e7
@the-other-james @HayleyO
Add newline
8d4109a
@asgibson @HayleyO
Update README.md
6e9acd2 
Fixed output running together on various lines that should be separated
@AlexKurek @HayleyO
Remove unused modules (nasa#137)
cffdacb
@the-other-james @HayleyO
Remove unused file (nasa#144)
a2b35df
@the-other-james @HayleyO
Switch back to csv parser (nasa#143)
ad09810
@the-other-james @HayleyO
Updated for new config format (nasa#140)
a22b8e6
@HayleyO
Allow access to OnAir from different directories
313fe70
@rachaelchertok1 @HayleyO
Updates to redis_adapter unit test and to redis_adapter.py to have su…
914064f 
…bscriptions only be set up if server ping was successful.
@rachaelchertok1 @HayleyO
All unit tests passing and 100% code coverage!
e16ef06
@rachaelchertok1 @HayleyO
Removed commented functions from test redis adapter code and removed …
1cb273f 
…debug prints from redis_adapter.py
@HayleyO
Merge branch 'unit_testing_134_create_multiple_redis_server_adapter' of
96a8f83 
https://github.com/HayleyO/OnAir_SummerInterns2024 into Issue_133_and_134_merge_branch
@HayleyO
Merge branch 'Issue_133_and_134_merge_branch' of https://github.com/H…
4e3b53b 
…ayleyO/OnAIR into Issue_133_and_134_merge_branch
@HayleyO
Fixed a weird git merge-caused error where code got repeated (my bad)
46c8333
@HayleyO HayleyO closed this Aug 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@HayleyO @rachaelchertok1 @dragonejt @asgibson @the-other-james @williamzhang0306 @AlexKurek