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: new page in trend explorer #2326

Closed
wants to merge 31 commits into from
Closed

feat: new page in trend explorer #2326

wants to merge 31 commits into from

Conversation

SaraVieira
Copy link
Contributor

@SaraVieira SaraVieira commented Sep 11, 2024 •
edited
Loading

  • Redoes the tables in the trend explorer to use tanstack table
  • Moves data fetching to its own hooks
  • Adds sorting to rag table
  • Small css tweaks in the signle query page
  • creates readonly version of json input

Screenshots:
image
image
image

related: #2059

SaraVieira and others added 26 commits September 6, 2024 02:44
@SaraVieira @skeptrunedev
fix getting started docs
48451b4
@SaraVieira @skeptrunedev
bugfix: improve start services and add tmuxp to README local Dev guide
a096999
@SaraVieira @skeptrunedev
cleanup: better CSS scoping for search component
5db43e2
@SaraVieira @skeptrunedev
chore: regenarate lock
0b7a8bb
@kosmoz @cdxker
ci: fix release workflow parse error
275de24
@cdxker
ci: changed RELEASE_PLEASE secret token name
093e1f1
@cdxker @densumesh
ci: add more commit flags to release please
325d917
@densumesh @cdxker
feature: add firecrawl to our docker compose
8e15ecf
@vid277 @skeptrunedev
bugfix: fix copy feature to allow for plain text in chat
8aec84e
@vid277 @skeptrunedev
cleanup: remove tooltip and globe icons
151bc0a
@drew-harris @skeptrunedev
fix: no datasets warning for chat
6a4a41a
@vid277 @skeptrunedev
fix: dataset warning for chat
cbb9ddd
@vid277 @skeptrunedev
fix: set topic list to empty if dataset is invalid
53491aa
@SaraVieira @skeptrunedev
fix: css fixes for rag page
fa1a97a
@SaraVieira @skeptrunedev
chore: fix eslint
15f4f8a
@SaraVieira @skeptrunedev
fix: add better margins to headings
a89293e
@cdxker @skeptrunedev
ci: release-please to use proper token
60e333c
@skeptrunedev @cdxker
cleanup: change default prompt such that LLM does not provide citatio…
9381197 
…ns by default
@SaraVieira
feat: new tables for rag queries
2b27e4f
@SaraVieira
fix: only open modal when there are results
574edde
@SaraVieira
chore: use memo
56033f3
@SaraVieira
reorg: move new components to shared lib
b6060fd
@SaraVieira
fix: move table code to the top of component
a6ee67e
@SaraVieira
feat: new data explorer table
3d57cac
@SaraVieira
fix: fill empty dates in rag usage chart
8eecbed
@SaraVieira
feat: add created at sort in rag
da2a9d3
@SaraVieira SaraVieira changed the base branch from main to rag_tables September 11, 2024 18:38
@SaraVieira SaraVieira requested review from skeptrunedev and removed request for drew-harris September 12, 2024 18:01
@SaraVieira SaraVieira marked this pull request as ready for review September 12, 2024 18:02
@skeptrunedev skeptrunedev force-pushed the rag_tables branch 3 times, most recently from 21aae65 to 3f1521d Compare September 13, 2024 08:20
@skeptrunedev
Copy link
Contributor

Closing in favor of - #2335

This was at a point where I didn't feel like I could practically cleanup the commit history so I cherry picked off into PR #2335

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@skeptrunedev skeptrunedev Awaiting requested review from skeptrunedev

@drew-harris drew-harris Awaiting requested review from drew-harris

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

Successfully merging this pull request may close these issues.
7 participants
@SaraVieira @skeptrunedev @kosmoz @cdxker @densumesh @vid277 @drew-harris