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.

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

Feature/enable tx context for micasense images #56

Merged
merged 5 commits into from
Sep 5, 2023

Conversation

saschadoemer
Copy link
Contributor

No description provided.

Enhancements have been made to clarify Javadocs in the 'ImageProcessingController' code, including the specification of the 'HTTP status' for image processing success. Additionally, 'transactionId' attribute has been added to the 'ImageProcessingRequest' class with its respective annotation and documentation. This provides clearer user response representation and better uniquely identifies each image processing request."
The change includes two major updates to the image processing workflow. Firstly, the addition of `transactionId` parameter to `processImage` method in `MicaSenseIntegrationService`, this allows tracking image processing requests as part of transactions. It improves data handling by explicitly associating each image with a unique transaction.

Secondly, the introduction of `ActiveMicaSenseTransactions` and `ActiveTransaction` classes for managing active image processing transactions. This enhances system's ability to track and manage active image processing requests, ensuring each request is correctly handled and processed."
Added a new method `endImageProcessing` in `ImageProcessingController` which marks the end of image processing for a given transactionId. This helps in effectively tracking and managing image processing jobs.

Also, this commit improves code readability by enhancing the class and method documentation across various classes like `MicaSenseIntegrationService` and `ActiveMicaSenseTransactions`. Changes include clarifying parameter definitions in methods and add informative comments for classes."
@saschadoemer
Copy link
Contributor Author

Closes #55

@saschadoemer saschadoemer added this pull request to the merge queue Sep 5, 2023
Merged via the queue into main with commit 4c1e293 Sep 5, 2023
1 check passed
@saschadoemer saschadoemer deleted the feature/enable-tx-context-for-micasense-images branch September 11, 2023 20:47
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.

None yet

1 participant