carloea2 commented on PR #8628: URL: https://github.com/apache/texera/pull/8628#issuecomment-5805292239
Thanks for the UP hackathon contribution, @Pinitsdv. Could you add side-by-side before/after screenshots using the same queries (an exact operator name and an intent such as "remove repeated rows"), plus a short GIF or recording of the initial loading state and keyword fallback when model loading fails? Please include browser, viewport, commit, and reproduction steps. The initialization-retry issue in my review still needs addressing; please include the regression test and exact test command/results with the update. For scope tracking, please keep this PR linked to the semantic-search portion of #8627, since that issue also proposes hover information and next-operator suggestions. The model-hosting and index-maintenance choices would benefit from community agreement on that issue before merging. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
