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

Fix CSV issues in client run #18

Merged
merged 4 commits into from
Apr 2, 2024
Merged

Fix CSV issues in client run #18

merged 4 commits into from
Apr 2, 2024

Conversation

darthtrevino
Copy link
Member

@darthtrevino darthtrevino commented Apr 2, 2024 •
edited
Loading

  • Handle when expected/configured columns are not present.
  • Push config token replacement entirely into run_pipeline_with_config
  • Set common default values for base_dirs in defaults.py instead fallbacks.
  • Fix SMT-reported issue where base-dir was not being injected into Blob configs

@darthtrevino darthtrevino requested a review from a team as a code owner April 2, 2024 18:16
@darthtrevino darthtrevino enabled auto-merge (squash) April 2, 2024 18:31
@darthtrevino darthtrevino disabled auto-merge April 2, 2024 18:41
@darthtrevino darthtrevino enabled auto-merge (squash) April 2, 2024 18:45
@darthtrevino darthtrevino merged commit 6a9ff04 into main Apr 2, 2024
4 checks passed
@darthtrevino darthtrevino deleted the fix/csv_load_issues branch April 2, 2024 19:02
logomachic pushed a commit to logomachic/graphrag that referenced this pull request Oct 31, 2024
@sirus-ms
Merge pull request microsoft#18 from prateejain-linked/KUSTO-4
3e484e0 
Using create_final_entities table rather than entity_description_embeddings & issue with to_csv not handling floats properly
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AlonsoGuevara AlonsoGuevara AlonsoGuevara approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@darthtrevino @AlonsoGuevara