Skip to content

Added image answering utility service#39

Open
vivekj-kore wants to merge 19 commits into
masterfrom
user/vivek/image_answering_utility
Open

Added image answering utility service#39
vivekj-kore wants to merge 19 commits into
masterfrom
user/vivek/image_answering_utility

Conversation

@vivekj-kore
Copy link
Copy Markdown

No description provided.

}

// If system not overloaded, process immediately
activeProcessingCount++;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we take activeProcessingCount using db for smooth transaction

}

async processPDF(filePath, include_base64 = false, uniqueId = null) {
const filename = path.basename(filePath);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utilise filename from previous method and pass it as parameter instead of creating multiple times

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

}
});

// Enhanced processing status endpoint
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove this status code from redis as it is already implemented in mongo code

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

ramgopals-kore
ramgopals-kore previously approved these changes Mar 13, 2025
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