Ingestion status callback update #142
Conversation
…essfully in Artemis
…essfully in Artemis
…essfully in Artemis
…ckUpdate # Conflicts: # app/pipeline/chat/course_chat_pipeline.py # app/pipeline/chat/exercise_chat_pipeline.py
WalkthroughThe recent changes streamline the data structures and functionalities within the application, focusing on lecture unit management. Key modifications include the removal of unnecessary fields in Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Webhook
participant DeletionPipelineWorker
participant StatusCallback
User->>Webhook: POST /lectures/delete
Webhook->>DeletionPipelineWorker: Trigger deletion process
DeletionPipelineWorker->>StatusCallback: Update deletion status
StatusCallback-->>DeletionPipelineWorker: Acknowledge status update
DeletionPipelineWorker-->>Webhook: Deletion process completed
Webhook-->>User: Response with deletion status
Recent review detailsConfiguration used: CodeRabbit UI Files selected for processing (1)
Additional comments not posted (4)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
Outside diff range, codebase verification and nitpick comments (2)
app/domain/ingestion/ingestion_pipeline_execution_dto.py (1)
11-11: Modification fromlecture_unitstolecture_unit.The change from
lecture_units(a list) tolecture_unit(a single object) has not been fully propagated throughout the codebase. Please update the following instances to ensure consistency:
app/pipeline/lecture_ingestion_pipeline.py
- Line:
self, lecture_units: list[LectureUnitDTO], artemis_base_url: str- Line:
for lecture_unit in lecture_units:app/web/routers/webhooks.py
- Line:
pipeline.delete_old_lectures(dto.lecture_units)app/domain/ingestion/deletionPipelineExecutionDto.py
- Line:
lecture_units: List[LectureUnitDTO] = Field(..., alias="pyrisLectureUnits")app/domain/data/metrics/competency_student_metrics_dto.py
- Line:
lecture_units: Dict[int, Set[int]] = Field({}, alias="lectureUnits")Please ensure that all logic handling these DTOs is updated accordingly to handle a single lecture unit instead of a list.
Analysis chain
Modification from
lecture_unitstolecture_unit.The change from
lecture_units(a list) tolecture_unit(a single object) indicates a significant shift in how the data is processed. Ensure that all logic handling this DTO is updated accordingly to handle a single lecture unit instead of a list.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify all instances where `lecture_units` was used and ensure they are updated to handle `lecture_unit`. # Test: Search for the usage of `lecture_units`. Expect: No instances of `lecture_units` should remain. rg --type python -A 5 $'lecture_units'Length of output: 68
Script:
#!/bin/bash # Description: Verify all instances where `lecture_units` was used and ensure they are updated to handle `lecture_unit`. # Test: Search for the usage of `lecture_units`. Expect: No instances of `lecture_units` should remain. rg --glob '*.py' -A 5 'lecture_units'Length of output: 2461
app/domain/data/lecture_unit_dto.py (1)
5-5:base_urlfield still present in the codebase.The
base_urlfield is still used in multiple files, indicating that its removal from the DTO structure might not be complete or other parts of the codebase still depend on it. Here are the specific locations wherebase_urlwas found:
app/web/status/LecturesDeletionStatusCallback.pyapp/web/status/IngestionStatusCallback.pyapp/web/status/status_update.pyapp/web/routers/webhooks.pyapp/web/routers/pipelines.pyapp/vector_database/lecture_schema.pyapp/retrieval/lecture_retrieval.pyapp/pipeline/lecture_ingestion_pipeline.pyapp/pipeline/chat/lecture_chat_pipeline.pyapp/pipeline/chat/exercise_chat_pipeline.pyapp/pipeline/chat/course_chat_pipeline.pyapp/llm/external/ollama.pyapp/domain/pipeline_execution_settings_dto.pyPlease ensure that any logic depending on the
base_urlfield is updated accordingly.Analysis chain
Removal of
to_updateandbase_urlfields.The removal of these fields simplifies the DTO structure. Ensure that any logic depending on these fields is updated accordingly.
Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify all instances where `to_update` and `base_url` were used and ensure they are removed or updated. # Test: Search for the usage of `to_update`. Expect: No instances of `to_update` should remain. rg --type python -A 5 $'to_update' # Test: Search for the usage of `base_url`. Expect: No instances of `base_url` should remain. rg --type python -A 5 $'base_url'Length of output: 128
Script:
#!/bin/bash # Description: Verify all instances where `to_update` and `base_url` were used and ensure they are removed or updated. # Test: Search for the usage of `to_update`. Expect: No instances of `to_update` should remain. rg 'to_update' --glob '*.py' -A 5 # Test: Search for the usage of `base_url`. Expect: No instances of `base_url` should remain. rg 'base_url' --glob '*.py' -A 5Length of output: 15912
There was a problem hiding this comment.
Actionable comments posted: 0
Outside diff range, codebase verification and nitpick comments (4)
app/web/routers/webhooks.py (3)
83-93: Testing required for the/lectures/deleteendpointThe
/lectures/deleteendpoint is not referenced in any test files, indicating a lack of automated test coverage. Ensure that this endpoint is properly tested to verify its functionality and security.
- Add unit tests for the
lecture_deletion_webhookfunction.- Ensure integration tests cover the
/lectures/deleteendpoint.Analysis chain
LGTM! Ensure proper usage and testing.
The endpoint
/lectures/deleteis well-defined and secured withTokenValidator.However, ensure that this endpoint is properly tested and used correctly in the codebase.
Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the usage of the `/lectures/delete` endpoint in the codebase. # Test: Search for the endpoint usage. Expect: Proper usage and testing. rg --type python -A 5 $'/lectures/delete'Length of output: 71
Script:
#!/bin/bash # Description: Verify the usage of the `/lectures/delete` endpoint in the codebase. # Test: Search for the endpoint usage. Expect: Proper usage and testing. rg --type py -A 5 '/lectures/delete'Length of output: 391
Script:
#!/bin/bash # Description: Verify the testing of the `/lectures/delete` endpoint in the codebase. # Test: Search for test cases related to the `/lectures/delete` endpoint. Expect: Proper testing. rg --type py -A 5 'test_lectures_delete'Length of output: 40
Script:
#!/bin/bash # Description: Verify the testing of the `/lectures/delete` endpoint in the codebase. # Test: Search for references to the `/lectures/delete` endpoint in test files. Expect: Proper testing. rg --type py '/lectures/delete' --glob '*test*'Length of output: 47
75-81: Inconsistent endpoint renaming detected.The endpoint
/lectures/fullIngestionstill exists in the codebase and should be renamed tolecture_ingestion_webhookfor consistency.
- File:
app/web/routers/webhooks.py
- Line: 75
Please update the old endpoint name to the new one to ensure uniformity across the codebase.
Analysis chain
Verify the endpoint renaming consistency.
The endpoint
/lectures/fullIngestionhas been renamed tolecture_ingestion_webhook.
Verify the endpoint renaming consistency.
The endpoint
/lectures/fullIngestionhas been renamed tolecture_ingestion_webhook.
Verify the endpoint renaming consistency.
The endpoint
/lectures/fullIngestionhas been renamed tolecture_ingestion_webhook.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the consistency of the endpoint renaming in the codebase. # Test: Search for the old and new endpoint names. Expect: No occurrences of the old name. rg --type python -A 5 $'lecture_webhook' rg --type python -A 5 $'lecture_ingestion_webhook'Length of output: 151
Script:
#!/bin/bash # Description: Verify the consistency of the endpoint renaming in the codebase. # Test: Search for the old and new endpoint names. Expect: No occurrences of the old name. rg --glob '*.py' -A 5 'lecture_webhook' rg --glob '*.py' -A 5 'lecture_ingestion_webhook'Length of output: 494
Script:
#!/bin/bash # Description: Verify the consistency of the endpoint renaming in the codebase. # Test: Search for the original endpoint name. Expect: No occurrences of the old name. rg --glob '*.py' -A 5 '/lectures/fullIngestion'Length of output: 411
51-68: No test cases found forrun_lecture_deletion_pipeline_worker.It is crucial to add proper test cases for the
run_lecture_deletion_pipeline_workerfunction to ensure its correct functionality, especially given its use of threading and exception handling.
- Ensure that this function is thoroughly tested.
- Add unit tests to cover various scenarios, including successful execution and exception handling.
Analysis chain
LGTM! Ensure proper usage and testing.
The function
run_lecture_deletion_pipeline_workeris well-structured and handles exceptions gracefully.However, ensure that this function is properly tested and used correctly in the codebase.
Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the usage of `run_lecture_deletion_pipeline_worker` in the codebase. # Test: Search for the function usage. Expect: Proper usage and testing. rg --type python -A 5 $'run_lecture_deletion_pipeline_worker'Length of output: 91
Script:
#!/bin/bash # Description: Verify the usage of `run_lecture_deletion_pipeline_worker` in the codebase. # Test: Search for the function usage. Expect: Proper usage and testing. rg --glob '*.py' -A 5 'run_lecture_deletion_pipeline_worker'Length of output: 593
Script:
#!/bin/bash # Description: Verify the testing of `run_lecture_deletion_pipeline_worker` in the codebase. # Test: Search for test cases related to the function. Expect: Proper testing. rg --glob '*test*.py' 'run_lecture_deletion_pipeline_worker'Length of output: 209
Script:
#!/bin/bash # Description: Verify the testing of `run_lecture_deletion_pipeline_worker` in the codebase. # Test: Search for test cases related to the function. Expect: Proper testing. rg 'run_lecture_deletion_pipeline_worker' --glob '*test*.py' --glob '*tests*.py' --glob '*_test.py' --glob '*_tests.py' --debugLength of output: 32901
Script:
#!/bin/bash # Description: Verify the testing of `run_lecture_deletion_pipeline_worker` in the codebase. # Test: Search for test cases related to the function without applying ignore rules. Expect: Proper testing. rg 'run_lecture_deletion_pipeline_worker' --glob '*test*.py' --glob '*tests*.py' --glob '*_test.py' --glob '*_tests.py' --no-ignoreLength of output: 280
app/pipeline/lecture_ingestion_pipeline.py (1)
297-317: Potential Issue withdelete_old_lecturesMethod CallThe method
delete_old_lecturesis called within therun_lecture_deletion_pipeline_workerfunction, but theartemis_base_urlparameter is not being passed. Ensure that theartemis_base_urlis correctly passed to the method to avoid runtime errors.
- File:
app/web/routers/webhooks.py- Function:
run_lecture_deletion_pipeline_worker- Line:
pipeline.delete_old_lectures(dto.lecture_units)Consider updating the method call to include the
artemis_base_urlparameter.Analysis chain
LGTM! Ensure proper usage and testing.
The method
delete_old_lecturesnow accepts parameters directly, enhancing reusability and decoupling from the class state.However, ensure that this method is properly tested and used correctly in the codebase.
Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the usage of the `delete_old_lectures` method in the codebase. # Test: Search for the method usage. Expect: Proper usage and testing. rg --type python -A 5 $'delete_old_lectures'Length of output: 74
Script:
#!/bin/bash # Description: Verify the usage of the `delete_old_lectures` method in the codebase. # Test: Search for the method usage. Expect: Proper usage and testing. rg --type py -A 5 $'delete_old_lectures'Length of output: 841
Script:
#!/bin/bash # Description: Verify the context of the `delete_old_lectures` method call in `app/web/routers/webhooks.py`. # Test: Search for the method call and include additional lines for context. Expect: Proper parameters passed. rg --type py -A 10 -B 10 $'pipeline.delete_old_lectures(dto.lecture_units)' app/web/routers/webhooks.pyLength of output: 104
Script:
#!/bin/bash # Description: Verify the context of the `delete_old_lectures` method call in `app/web/routers/webhooks.py`. # Test: Search for the method call and include additional lines for context. Expect: Proper parameters passed. rg --type py -A 20 -B 20 'delete_old_lectures' app/web/routers/webhooks.pyLength of output: 1391
There was a problem hiding this comment.
Actionable comments posted: 0
Outside diff range, codebase verification and nitpick comments (3)
app/web/status/LecturesDeletionStatusCallback.py (1)
1-2: Remove unnecessary blank lines.The blank line after the import statement is unnecessary and can be removed to improve readability.
-app/pipeline/lecture_ingestion_pipeline.py (2)
95-95: Add checks for optionaldtoparameter.The
dtoparameter is used in several places without checking if it isNone. Sincedtois now optional, please add appropriate checks to handle cases wheredtomight beNoneto avoid potentialAttributeErrorexceptions.
- Lines where
dtois accessed without checks:
app/pipeline/lecture_ingestion_pipeline.py:98app/pipeline/lecture_ingestion_pipeline.py:99app/pipeline/lecture_ingestion_pipeline.py:100app/pipeline/lecture_ingestion_pipeline.py:101app/pipeline/lecture_ingestion_pipeline.py:106app/pipeline/lecture_ingestion_pipeline.py:111app/pipeline/lecture_ingestion_pipeline.py:112app/pipeline/lecture_ingestion_pipeline.py:121Analysis chain
Constructor update approved.
The constructor now accepts an optional
dtoparameter, enhancing flexibility.Ensure that the rest of the code correctly handles the optional
dto.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the usage of `dto` in the codebase to ensure it handles the optional parameter correctly. # Test: Search for the usage of `dto`. Expect: Proper handling of optional `dto`. rg --type python -A 5 $'self.dto'Length of output: 63
Script:
#!/bin/bash # Description: Verify the usage of `dto` in the codebase to ensure it handles the optional parameter correctly. # Test: Search for the usage of `dto`. Expect: Proper handling of optional `dto`. rg --type py -A 5 $'self.dto'Length of output: 2806
130-138: Ensureself.dtois notNonebefore accessing its properties in the__call__method.The
__call__method inapp/pipeline/lecture_ingestion_pipeline.pyaccessesself.dtoproperties directly without checking ifself.dtoisNone. This could lead to potential runtime errors ifself.dtoisNone.
- Add a check to ensure
self.dtois notNonebefore accessing its properties in the__call__method.Analysis chain
Simplified lecture chunking and interpretation.
The lecture chunking and interpretation process has been simplified by directly accessing properties from
self.dto.lecture_unit. Ensure thatself.dtois notNonebefore accessing its properties.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify that `self.dto` is not `None` before accessing its properties in the `__call__` method. # Test: Search for the usage of `self.dto` in the `__call__` method. Expect: Proper null checks. rg --type python -A 5 $'self.dto' | rg -B 5 'if self.dto is not None'Length of output: 99
Script:
#!/bin/bash # Description: Verify that `self.dto` is not `None` before accessing its properties in the `__call__` method. # Test: Search for the usage of `self.dto` in the `__call__` method. Expect: Proper null checks. rg -t py -A 5 $'self.dto' | rg -B 5 'if self.dto is not None'Length of output: 60
Script:
#!/bin/bash # Description: Verify that `self.dto` is not `None` before accessing its properties in the `__call__` method in `app/pipeline/lecture_ingestion_pipeline.py`. # Locate the `__call__` method in the specified file. ast-grep --lang python --pattern $'def __call__($_):\n $$$' app/pipeline/lecture_ingestion_pipeline.pyLength of output: 105
Script:
#!/bin/bash # Description: Verify the presence of the `__call__` method and check for `None` validation for `self.dto` in `app/pipeline/lecture_ingestion_pipeline.py`. # Locate the `__call__` method in the specified file. rg -A 10 'def __call__' app/pipeline/lecture_ingestion_pipeline.pyLength of output: 580
There was a problem hiding this comment.
Actionable comments posted: 0
Outside diff range, codebase verification and nitpick comments (2)
app/pipeline/lecture_ingestion_pipeline.py (2)
95-97: Ensure proper handling ofNonevalue fordto.The
dtoparameter is now optional, but there are multiple instances in the code whereself.dtois accessed without checking forNone. This can lead toNoneTypeerrors. Please ensure that all usages ofself.dtowithin the class handle the possibility of it beingNone.
app/pipeline/lecture_ingestion_pipeline.py: Lines whereself.dtois accessed without checks.Analysis chain
Ensure proper handling of
Nonevalue fordto.The
dtoparameter is now optional. Ensure that all usages ofself.dtowithin the class handle the possibility of it beingNoneto avoidNoneTypeerrors.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify that all usages of `self.dto` handle the possibility of it being `None`. # Test: Search for `self.dto` usage. Expect: Proper handling of `None` values. rg --type python 'self\.dto'Length of output: 61
Script:
#!/bin/bash # Description: Verify that all usages of `self.dto` handle the possibility of it being `None`. # Test: Search for `self.dto` usage. Expect: Proper handling of `None` values. rg --type-add 'py:*.py' --type py 'self\.dto'Length of output: 914
297-317: Thedelete_old_lecturesmethod call is missing a required parameter.The method call to
delete_old_lecturesinapp/web/routers/webhooks.pyonly passesdto.lecture_unitsand does not provide the requiredartemis_base_urlparameter. This needs to be corrected to ensure the method functions as intended.
- File:
app/web/routers/webhooks.py- Line: Around 5 lines after the method call
pipeline.delete_old_lectures(dto.lecture_units)Please update the method call to include the
artemis_base_urlparameter.Analysis chain
LGTM! Verify the method usage in the codebase.
The changes to the
delete_old_lecturesmethod enhance its reusability and clarity.However, ensure that all calls to
delete_old_lecturespass the correct parameters.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify that all calls to `delete_old_lectures` pass the correct parameters. # Test: Search for the method usage. Expect: Correct parameters passed. rg --type python 'delete_old_lectures'Length of output: 69
Script:
#!/bin/bash # Description: Verify that all calls to `delete_old_lectures` pass the correct parameters. # Test: Search for the method usage. Expect: Correct parameters passed. rg --type py 'delete_old_lectures'Length of output: 190
Script:
#!/bin/bash # Description: Verify the exact context of the `delete_old_lectures` method call in `app/web/routers/webhooks.py`. # Test: Display the relevant lines around the method call. rg --type py -A 5 -B 5 'pipeline.delete_old_lectures' app/web/routers/webhooks.pyLength of output: 479
There was a problem hiding this comment.
Actionable comments posted: 2
Outside diff range, codebase verification and nitpick comments (5)
app/web/status/lecture_deletion_status_callback.py (1)
12-31: Well-implemented class for deletion status updates.The
LecturesDeletionStatusCallbackclass is well-implemented with dynamic stage management and URL construction. However, consider adding error handling for potential issues with thebase_urlformat or network problems during the callback.Consider implementing error handling mechanisms to improve robustness, such as:
try: # existing code except Exception as e: logger.error(f"Failed to update status: {e}") raiseapp/web/status/ingestion_status_callback.py (1)
Line range hint
14-43: Enhanced functionality for lecture-specific ingestion status updates.The
IngestionStatusCallbackclass has been effectively updated to includelecture_unit_id, enhancing its functionality. The docstring update is also appropriate. Consider adding error handling for potential issues with thelecture_unit_idor network problems during the callback.Consider implementing error handling mechanisms to improve robustness, such as:
try: # existing code except Exception as e: logger.error(f"Failed to update status: {e}") raiseapp/web/routers/webhooks.py (2)
75-81: Approve implementation; suggest documentation improvements.The implementation of
lecture_ingestion_webhookis appropriate, using a separate thread for non-blocking operations. Consider improving the documentation to better reflect the specific purpose of this webhook, which is to trigger the lecture ingestion pipeline.
83-93: Approve implementation; suggest documentation improvements.The implementation of
lecture_deletion_webhookis consistent with other webhook functions, using a separate thread for non-blocking operations. Consider improving the documentation to better reflect the specific purpose of this webhook, which is to trigger the lecture deletion process.app/pipeline/lecture_ingestion_pipeline.py (1)
298-318: Refactoring ofdelete_old_lecturesapproved.The method's refactoring to accept parameters directly enhances its reusability and clarity. This change supports the PR's objectives effectively.
Consider adding more detailed logging for each deletion attempt to improve traceability and debugging.
logger.info(f"Attempting to delete lecture: {lecture_unit.lecture_unit_id}") if self.delete_lecture_unit( lecture_unit.course_id, lecture_unit.lecture_id, lecture_unit.lecture_unit_id, artemis_base_url, ): logger.info("Lecture deleted successfully") else: logger.error("Failed to delete lecture")
| lecture_unit: LectureUnitDTO = Field(..., alias="pyrisLectureUnit") | ||
| settings: Optional[PipelineExecutionSettingsDTO] | ||
| initial_stages: Optional[List[StageDTO]] = Field( | ||
| default=None, alias="initialStages" | ||
|
|
There was a problem hiding this comment.
Refactor required for missing imports and type annotations.
The class IngestionPipelineExecutionDto has been updated to handle a single lecture_unit and added new attributes settings and initial_stages. However, there are several issues:
- The
Optionaltype is used but not imported fromtyping. PipelineExecutionSettingsDTOandStageDTOare referenced but not defined or imported.
To resolve these issues, add the necessary imports and definitions:
+from typing import Optional
+from app.domain.settings import PipelineExecutionSettingsDTO
+from app.domain.stages import StageDTOCommittable suggestion was skipped due to low confidence.
Tools
Ruff
11-11: Undefined name
Optional(F821)
11-11: Undefined name
PipelineExecutionSettingsDTO(F821)
12-12: Undefined name
Optional(F821)
12-12: Undefined name
StageDTO(F821)
| self.delete_lecture_unit( | ||
| self.dto.lecture_unit.course_id, | ||
| self.dto.lecture_unit.lecture_id, | ||
| self.dto.lecture_unit.lecture_unit_id, | ||
| self.dto.settings.artemis_base_url, | ||
| ) |
There was a problem hiding this comment.
Streamlined __call__ method approved.
The method has been effectively simplified to focus on a single lecture_unit, which aligns with the updated constructor and enhances clarity. The integration of the new callback mechanism is well-handled.
Consider adding error handling for potential None values in dto to prevent runtime errors.
if self.dto is not None:
# existing code
else:
logger.error("DTO is None, cannot proceed with ingestion.")
return FalseAlso applies to: 131-139
| lecture_unit: LectureUnitDTO = Field(..., alias="pyrisLectureUnit") | ||
| settings: Optional[PipelineExecutionSettingsDTO] | ||
| initial_stages: Optional[List[StageDTO]] = Field( | ||
| default=None, alias="initialStages" |
There was a problem hiding this comment.
Add missing imports and address type annotations.
The class IngestionPipelineExecutionDto has been updated with new attributes that require additional imports to resolve the issues flagged by static analysis tools:
- The
Optionaltype is used but not imported fromtyping. PipelineExecutionSettingsDTOandStageDTOare referenced but not defined or imported.
To resolve these issues, add the necessary imports and definitions:
+from typing import Optional
+from app.domain.settings import PipelineExecutionSettingsDTO
+from app.domain.stages import StageDTOThese changes will ensure that the code is syntactically correct and that the references to other DTOs are properly resolved.
Committable suggestion was skipped due to low confidence.
Tools
Ruff
11-11: Undefined name
Optional(F821)
11-11: Undefined name
PipelineExecutionSettingsDTO(F821)
12-12: Undefined name
Optional(F821)
12-12: Undefined name
StageDTO(F821)
This PR modifies the ingestion Stucture to be able to update artemis successfully, when ingestion is finished.
Summary by CodeRabbit
New Features
Improvements
Bug Fixes