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.

Sign up for GitHub

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

Jump to bottom

Feat/gpt4v (wip) #526

Closed
wants to merge 75 commits into from
Closed

Feat/gpt4v (wip) #526

wants to merge 75 commits into from

Conversation

abrichr
Copy link
Member

@abrichr abrichr commented Nov 21, 2023

Work-in-progress

Addresses #513

Mustaballer and others added 30 commits June 5, 2023 13:51
@Mustaballer
Copy from closed PR magic wormhole working code and test cases
8c0a78d
@Mustaballer
Merge branch 'main' into share-magic-wormhole
b4b7879
@Mustaballer
Merge branch 'main' into share-magic-wormhole
395fa00
@Mustaballer
Merge remote-tracking branch 'upstream/main' into share-magic-wormhole
5fbdc1a
@Mustaballer
Merge branch 'share-magic-wormhole' of https://github.com/Mustaballer…
0a0208e 
…/puterbot into share-magic-wormhole
@Mustaballer
modify export_sql to use paramerterized queries to prevent sql injection
9dc1850
@Mustaballer
try resolve merge conflict with poetry.lock
53f11f5
@Mustaballer
Merge branch 'main' into share-magic-wormhole
0977859
@Mustaballer
fix merge conflict and use better approach for overwriting env
cbee5e8
@Mustaballer
Remove unnecessary function and pass test cases
9819330
@Mustaballer
reformat file and group constants together in config.py
afd9810
@Mustaballer
ran black
2211f78
@Mustaballer
ran black
f58fd5f
@Mustaballer
moved functions in crud.py to db.py
dafca05
@Mustaballer
use tempfile in test_share.py and address minor changes
3434a00
@Mustaballer
restore original db name instead of literal
b801caf
@Mustaballer
ran black -l 60, and it used call chain
4c59c10
@0dm
Merge branch 'main' into share-magic-wormhole
088f370
@Mustaballer
Merge branch 'main' into share-magic-wormhole
1096a15
@Mustaballer
Add exception handling when ctrl+c during sharing that deletes db and…
01eef24 
… zip file and reverts db environment. Also add type and return annotations
@Mustaballer
ran black and update poetry.lock
5e52b5c
@Mustaballer
Add missing docstring in db.py
8317634
@Mustaballer
delete temp .env if ctrl+c during sharing
e675227
@Mustaballer
merge with latest main
04c69b8
@Mustaballer
Add .env.example and generate env in config.py
2d54e83
@Mustaballer
use .env.example for creating .env and removed unnecessary exceptions
3992d1c
@Mustaballer
Merge branch 'main' into share-magic-wormhole
d4d3f1c
@Mustaballer
use new approach for copying db
2f2c0dd
@Mustaballer
Merge branch 'share-magic-wormhole' of https://github.com/Mustaballer…
7a67386 
…/puterbot into share-magic-wormhole
@Mustaballer
modify copy deb function to return recording data and remov comments …
6196876 
…and unused set_db_url
Mustaballer and others added 27 commits August 3, 2023 19:24
@Mustaballer
Merge branch 'share-magic-wormhole' of https://github.com/Mustaballer…
b6cce11 
…/puterbot into share-magic-wormhole
@Mustaballer
run black --preview and modify main.yml to check black --preview
d9eefa0
@Mustaballer
remove unused import
961130b
@Mustaballer
Add timestamp to exported recording db files and update unit tests
5fd9868
@Mustaballer
Merge branch 'main' into share-magic-wormhole
453d2c2
@Mustaballer
update poetry.lock
4d66584
@Mustaballer
Merge branch 'main' into share-magic-wormhole
75b324d
@Mustaballer
fix: enhance publish action and authors in pyproject.toml
dc5f2b6
@Mustaballer
modify release-and-publish.yml
ebea2cb
@Mustaballer
change author name to OpenAdapt.AI Team
c77f339
@Mustaballer
Merge remote-tracking branch 'upstream/enhance-publishing' into share…
8b1299d 
…-magic-wormhole
@Mustaballer
test publish to test pypi
09dd4d6
@Mustaballer
fix poetry conflicts and conflicting files
8ea0ee4
@Mustaballer
Merge branch 'main' into share-magic-wormhole
1b2eef3
@Mustaballer
resolve merge conflicts and linting errors from recent merge
0e8d620
@Mustaballer
Merge branch 'main' into share-magic-wormhole
5b9bbe0
@Mustaballer
Merge branch 'main' into share-magic-wormhole
24a3e79
@Mustaballer
update poetry.lock file and some formatting
a03bd59
@Mustaballer
fix wormhole sharing
a94c303
@Mustaballer
Merge remote-tracking branch 'upstream/main' into share-magic-wormhole
ca04296
@abrichr
poetry lock
61553f0
@abrichr
fix failing tests
22b5f0c
@abrichr
Merge branch 'main' into share-magic-wormhole
00c79cf
@abrichr
poetry lock
21340b2
@abrichr
add spacy-curated-transformers
0edcd88
@abrichr
wip
4b8b8b0
@abrichr
wip
d321ba7
@sweep-ai Sweep AI
Copy link

sweep-ai bot commented Nov 21, 2023

Apply Sweep Rules to your PR?

  • Apply: All new business logic should have corresponding unit tests.
  • Apply: Refactor large functions to be more modular.
  • Apply: Add docstrings to all functions and file headers.

@abrichr
Copy link
Member Author

abrichr commented Nov 21, 2023

Moved to #527

@abrichr abrichr closed this Nov 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@abrichr @Mustaballer @0dm