Commit graph

3412 commits

Author SHA1 Message Date
Igor Ilic
920bc78f15 refactor: Remove unused code 2025-09-27 00:18:57 +02:00
Geoff-Robin
2921021ca3 improved code readability by splitting code blocks under conditional statements into separate functions 2025-09-27 00:16:44 +02:00
Geoff-Robin
656894370e Edited test_relation_db_migration.py to include schema_only ingestion testcase 2025-09-27 00:16:44 +02:00
Geoff-Robin
67f948a145 solved nitpick comments 2025-09-27 00:16:44 +02:00
Geoff-Robin
e7bcf9043f solved more nitpick comments 2025-09-27 00:16:44 +02:00
Geoff-Robin
df8b80d4a9 solved more nitpick comments 2025-09-27 00:16:44 +02:00
Geoff-Robin
1e59f1594c solved more nitpick comments 2025-09-27 00:16:44 +02:00
Geoff-Robin
93c733e687 solved more nitpick comments 2025-09-27 00:16:44 +02:00
Geoff-Robin
7ec066111e Solved address configuration key inconsistency. 2025-09-27 00:16:44 +02:00
Geoff-Robin
60016a6b09 more nitpick comments solved 2025-09-27 00:16:44 +02:00
Geoff-Robin
7cf4a0daeb id mismatch risk negated 2025-09-27 00:16:44 +02:00
Geoff-Robin
1ba9e1df31 done with ruff checks 2025-09-27 00:16:44 +02:00
Geoff-Robin
51dfac359d Removed print statements used while debugging 2025-09-27 00:16:44 +02:00
Geoff-Robin
f5bb91e49d added description attribute to every schema model 2025-09-27 00:16:44 +02:00
Geoff-Robin
17df14363c integrated schema only ingestion 2025-09-27 00:16:44 +02:00
Geoff-Robin
a43f19cc59 ingest_database_schema with a slight alteration with return value as Dict[str,List[DataPoint] | DataPoint]] 2025-09-27 00:16:44 +02:00
Geoff-Robin
9d801f5fe0 Done creating models.py and ingest_database_schema.py 2025-09-27 00:16:44 +02:00
Boris
726d4d8535
fix: limit onnxruntime version (#1473)
<!-- .github/pull_request_template.md -->

## Description
<!--
Please provide a clear, human-generated description of the changes in
this PR.
DO NOT use AI-generated descriptions. We want to understand your thought
process and reasoning.
-->

## Type of Change
<!-- Please check the relevant option -->
- [ ] Bug fix (non-breaking change that fixes an issue)
- [ ] New feature (non-breaking change that adds functionality)
- [ ] Breaking change (fix or feature that would cause existing
functionality to change)
- [ ] Documentation update
- [ ] Code refactoring
- [ ] Performance improvement
- [ ] Other (please specify):

## Screenshots/Videos (if applicable)
<!-- Add screenshots or videos to help explain your changes -->

## Pre-submission Checklist
<!-- Please check all boxes that apply before submitting your PR -->
- [ ] **I have tested my changes thoroughly before submitting this PR**
- [ ] **This PR contains minimal changes necessary to address the
issue/feature**
- [ ] My code follows the project's coding standards and style
guidelines
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [ ] I have added necessary documentation (if applicable)
- [ ] All new and existing tests pass
- [ ] I have searched existing PRs to ensure this change hasn't been
submitted already
- [ ] I have linked any relevant issues in the description
- [ ] My commits have clear and descriptive messages

## DCO Affirmation
I affirm that all code in every commit of this pull request conforms to
the terms of the Topoteretes Developer Certificate of Origin.
2025-09-26 10:21:20 +02:00
Boris Arzentar
1deab2d54e
fix: limit onnxruntime version 2025-09-26 09:57:53 +02:00
Vasilije
8246a6a02f
fix: Remove creation of default user during search (#1455)
<!-- .github/pull_request_template.md -->

## Description
Removed default user creation during brute force search. Even when a
user is provided to search it's not forwarded to the Retrievers, the
retrievers always created a default user and sent telemetry as the
default user which is inaccurate, they also create a default user even
when there shouldn't be one.

if this information is necessary for telemetry we should forward the
user information that was sent through search through the retrievers and
not always create a default user

## Type of Change
<!-- Please check the relevant option -->
- [ ] Bug fix (non-breaking change that fixes an issue)
- [ ] New feature (non-breaking change that adds functionality)
- [x] Breaking change (fix or feature that would cause existing
functionality to change)
- [ ] Documentation update
- [ ] Code refactoring
- [ ] Performance improvement
- [ ] Other (please specify):

## Changes Made
Removed user as parameter from brute force search, removed default user
creation that was supplied as parameter to brute force search

## Testing
Ran simple example, waiting for CI/CD results

## Pre-submission Checklist
<!-- Please check all boxes that apply before submitting your PR -->
- [x] **I have tested my changes thoroughly before submitting this PR**
- [x] **This PR contains minimal changes necessary to address the
issue/feature**
- [x] My code follows the project's coding standards and style
guidelines
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [ ] I have added necessary documentation (if applicable)
- [ ] All new and existing tests pass
- [x] I have searched existing PRs to ensure this change hasn't been
submitted already
- [ ] I have linked any relevant issues in the description
- [x] My commits have clear and descriptive messages

## DCO Affirmation
I affirm that all code in every commit of this pull request conforms to
the terms of the Topoteretes Developer Certificate of Origin.
2025-09-25 21:11:42 +02:00
Vasilije
9b1d2bfdf9
feat: cors issue if frontend runs on any port other than 3000 (#1457)
<!-- .github/pull_request_template.md -->

## Description
<!-- 
Please provide a clear, human-generated description of the changes in
this PR.
DO NOT use AI-generated descriptions. We want to understand your thought
process and reasoning.
-->

## Type of Change
<!-- Please check the relevant option -->
- [ ] Bug fix (non-breaking change that fixes an issue)
- [ ] New feature (non-breaking change that adds functionality)
- [ ] Breaking change (fix or feature that would cause existing
functionality to change)
- [ ] Documentation update
- [ ] Code refactoring
- [ ] Performance improvement
- [ ] Other (please specify):

## Changes Made
<!-- List the specific changes made in this PR -->
- 
- 
- 

## Testing
<!-- Describe how you tested your changes -->

## Screenshots/Videos (if applicable)
<!-- Add screenshots or videos to help explain your changes -->

## Pre-submission Checklist
<!-- Please check all boxes that apply before submitting your PR -->
- [ ] **I have tested my changes thoroughly before submitting this PR**
- [ ] **This PR contains minimal changes necessary to address the
issue/feature**
- [ ] My code follows the project's coding standards and style
guidelines
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [ ] I have added necessary documentation (if applicable)
- [ ] All new and existing tests pass
- [ ] I have searched existing PRs to ensure this change hasn't been
submitted already
- [ ] I have linked any relevant issues in the description
- [ ] My commits have clear and descriptive messages

## Related Issues
<!-- Link any related issues using "Fixes #issue_number" or "Relates to
#issue_number" -->

## Additional Notes
<!-- Add any additional notes, concerns, or context for reviewers -->

## DCO Affirmation
I affirm that all code in every commit of this pull request conforms to
the terms of the Topoteretes Developer Certificate of Origin.
2025-09-25 21:11:01 +02:00
Boris
013baa3bfc
Merge branch 'dev' into remove-default-user-search 2025-09-25 21:02:00 +02:00
Boris
09fc566504
Merge branch 'dev' into feature/cog-2728-bug-cors-issue-if-frontend-runs-on-any-port-other-than-3000 2025-09-25 21:01:50 +02:00
Igor Ilic
f6254aa5fa
fix: Resolve issue with only_context [COG-3032] (#1452)
<!-- .github/pull_request_template.md -->

## Description
Resolve issue of only context search not working without backend access
control enabled and not being forwarded when backend access control is
enabled

## Type of Change
<!-- Please check the relevant option -->
- [x] Bug fix (non-breaking change that fixes an issue)
- [ ] New feature (non-breaking change that adds functionality)
- [ ] Breaking change (fix or feature that would cause existing
functionality to change)
- [ ] Documentation update
- [ ] Code refactoring
- [ ] Performance improvement
- [ ] Other (please specify):

## Testing
Tested in local SaaS by calling search endpoint on different datasets
with different parameters

## Pre-submission Checklist
<!-- Please check all boxes that apply before submitting your PR -->
- [x] **I have tested my changes thoroughly before submitting this PR**
- [x] **This PR contains minimal changes necessary to address the
issue/feature**
- [x] My code follows the project's coding standards and style
guidelines
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [ ] I have added necessary documentation (if applicable)
- [x] All new and existing tests pass
- [x] I have searched existing PRs to ensure this change hasn't been
submitted already
- [ ] I have linked any relevant issues in the description
- [x] My commits have clear and descriptive messages

## Related Issues
Fixes issue #COG-3032

## DCO Affirmation
I affirm that all code in every commit of this pull request conforms to
the terms of the Topoteretes Developer Certificate of Origin.
2025-09-25 20:58:33 +02:00
Boris
ad694ff9fe
Merge branch 'dev' into remove-default-user-search 2025-09-25 18:53:09 +02:00
Boris
f041589d20
Merge branch 'dev' into feature/cog-2728-bug-cors-issue-if-frontend-runs-on-any-port-other-than-3000 2025-09-25 18:53:06 +02:00
Boris
b50556b877
Merge branch 'dev' into fix-only-context 2025-09-25 18:53:03 +02:00
Boris
668dd933ff
chore: Limit pylance to 0.36 because of intel MacOS13 (#1468)
<!-- .github/pull_request_template.md -->

## Description
Latest pylance version 0.37 is not supported for intel architecture
MacOS versions, limit pylance version until MacOS13 can be deprecated
for support

Similar situation is happening with ruff with their new release 10min
ago, it's why it's also limited in this PR

## Type of Change
<!-- Please check the relevant option -->
- [x] Bug fix (non-breaking change that fixes an issue)
- [ ] New feature (non-breaking change that adds functionality)
- [ ] Breaking change (fix or feature that would cause existing
functionality to change)
- [ ] Documentation update
- [ ] Code refactoring
- [ ] Performance improvement
- [ ] Other (please specify):

## Pre-submission Checklist
<!-- Please check all boxes that apply before submitting your PR -->
- [x] **I have tested my changes thoroughly before submitting this PR**
- [x] **This PR contains minimal changes necessary to address the
issue/feature**
- [x] My code follows the project's coding standards and style
guidelines
- [x] I have added tests that prove my fix is effective or that my
feature works
- [ ] I have added necessary documentation (if applicable)
- [x] All new and existing tests pass
- [x] I have searched existing PRs to ensure this change hasn't been
submitted already
- [x] I have linked any relevant issues in the description
- [x] My commits have clear and descriptive messages

## DCO Affirmation
I affirm that all code in every commit of this pull request conforms to
the terms of the Topoteretes Developer Certificate of Origin.
2025-09-25 18:52:45 +02:00
Igor Ilic
f09376429b refactor: Remove telemetry call 2025-09-25 17:39:29 +02:00
Igor Ilic
3b9415ee88 test: Resolve failing unit tests 2025-09-25 17:27:11 +02:00
Igor Ilic
f2edfaa9b9 refactor: Add scikit learn for evals 2025-09-25 17:23:14 +02:00
Igor Ilic
88655031ff chore: Remove scikit dependency 2025-09-25 17:14:56 +02:00
Boris
ade412cd9e
Merge branch 'dev' into remove-default-user-search 2025-09-25 17:10:15 +02:00
Boris
2e8e02bf59
Merge branch 'dev' into feature/cog-2728-bug-cors-issue-if-frontend-runs-on-any-port-other-than-3000 2025-09-25 17:09:35 +02:00
Igor Ilic
71e1070820 Merge branch 'dev' into pylance-fix 2025-09-25 17:08:28 +02:00
Boris
91f21247e0
Merge branch 'dev' into fix-only-context 2025-09-25 17:06:39 +02:00
Igor Ilic
bb0ae06a0a Merge branch 'dev' of github.com:topoteretes/cognee into dev 2025-09-25 17:04:50 +02:00
Igor Ilic
bcc1747ab8 chore: resolve ruff v0.13.2 support issue 2025-09-25 17:02:58 +02:00
Vasilije
997b85e1ce
fix: Cog 2826 clean up poetry (#1305)
<!-- .github/pull_request_template.md -->

## Description
<!-- Provide a clear description of the changes in this PR -->

## DCO Affirmation
I affirm that all code in every commit of this pull request conforms to
the terms of the Topoteretes Developer Certificate of Origin.
2025-09-25 16:56:26 +02:00
Igor Ilic
cf3f5945e7 chore: Limit pylance to 0.36 for MacOS13 2025-09-25 16:44:55 +02:00
Igor Ilic
4054307b15 refactor: Remove comment 2025-09-25 16:03:11 +02:00
Igor Ilic
50032dd133 fix: install aws for gh action 2025-09-25 16:02:30 +02:00
Igor Ilic
664459e239 refactor: Install baml only for BAML test 2025-09-25 15:30:27 +02:00
Igor Ilic
61ef6fa444 chore: Update pyproject 2025-09-25 15:26:10 +02:00
Igor Ilic
6f8f9bf7de refactor: make comment more understandable 2025-09-25 13:58:52 +02:00
Igor Ilic
8265ec0334 refactor: Add missing install info 2025-09-25 13:57:14 +02:00
Igor Ilic
d1724c710b refactor: Add proper pip install command for optional extras 2025-09-25 13:55:01 +02:00
Igor Ilic
ca2e63bd84 refactor: Move postgres handling to database creation time 2025-09-25 13:49:04 +02:00
Igor Ilic
d2d0d0de4e refactor: install cognee defined baml version for CI/CD 2025-09-25 13:32:09 +02:00
Igor Ilic
8cbc3eb877 Merge branch 'dev' into COG-2826 2025-09-25 13:31:21 +02:00