Merged
Conversation
lossyrob
force-pushed
the
feature/chat
branch
2 times, most recently
from
933a494 to
e70cc0b
Compare
lossyrob
force-pushed
the
feature/chat
branch
from
e70cc0b to
6035766
Compare
This was causing exceptions not to be logged, which made debugging more difficult.
Fixes error starting server from vs code.
Previously it was pointed to my branch, but wasn't actually using the postgres vector store, so we can use an official release.
lossyrob
force-pushed
the
feature/chat
branch
from
6035766 to
179ad39
Compare
This solves issue with running in dev mode with vs code extension requiring copying ossdbtoolsservice_main.py out to the top level of the repo.
lossyrob
force-pushed
the
feature/chat
branch
2 times, most recently
from
3563ecb to
6247560
Compare
Main motivation here is to create separate code paths for pyinstaller if using a web server vs json rpc server. The pyinstaller targeting vscode was including ssl, which produced some issues on mac osx arm in loading openssl shared libraries. These were only required for the web server, and this refactor produces different code paths for web server vs rpc server context such that ssl and other web server dependencies will not be included in the pyinstaller.
lossyrob
force-pushed
the
feature/chat
branch
from
718d0b5 to
febc6b2
Compare
lossyrob
force-pushed
the
feature/chat
branch
from
febc6b2 to
6d82cbb
Compare
lossyrob
force-pushed
the
feature/chat
branch
from
509e193 to
45b1a65
Compare
lossyrob
force-pushed
the
feature/chat
branch
from
6196710 to
3bcf7cc
Compare
lossyrob
force-pushed
the
feature/chat
branch
from
3bcf7cc to
87dd188
Compare
mmcfarland
approved these changes
Feb 8, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds a new ChatService to pgtoolsservice, which enables Github Copilot integration.
This is based off of the branch in #500 . This should be updated once #500 merges.
Some other improvements/fixes were made, which are captured in separate commits.