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

Ingestion visualization #15

Closed
wants to merge 2 commits into from
Closed

Ingestion visualization #15

wants to merge 2 commits into from

Conversation

therealslimjp
Copy link

POC. If you got another ideas on what i should add, feel free to leave a comment

@therealslimjp therealslimjp force-pushed the ingestion-visualization branch 3 times, most recently from 23c00e6 to 0574b46 Compare August 31, 2024 11:23
@therealslimjp
initial
0933e7a 
Signed-off-by: therealslimjp <[email protected]>
@therealslimjp
show size
bdcddec 
Signed-off-by: therealslimjp <[email protected]>
@rakeshJn
Copy link
Member

rakeshJn commented Sep 3, 2024

Thanks, will take a look soon.

jurossiar
jurossiar reviewed Sep 4, 2024
View reviewed changes
lake_viewer.py
dotenv.load_dotenv(dotenv.find_dotenv(usecwd=True)) #Use current working directory to load .env file

print(os.environ)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, remove this print

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ofc. Forgot to take it out. Debugging…

@jurossiar
Copy link
Collaborator

Seems Granularity is not working as expected. I think should be grouped by hour, day, week in the X axis
E.g.:
image
image
image
image

@rakeshJn
Copy link
Member

Hi @therealslimjp , sorry it took a while with all the re-architecture being done. In the new set up, I have tried to capture what you were trying to achieve through this PR. Please check: #45
I have merged it to main so you can try. It does open a lot of other possibilities for insights I think.
Please try out the new one, it is much cleaner as we can separate out python side as pure rest api and front end side in Svelte/JS.
I will close this PR. Also there are several issues open if you want to pick any of those..... Thanks.

@rakeshJn rakeshJn closed this Oct 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jurossiar jurossiar jurossiar left review comments

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
@therealslimjp @rakeshJn @jurossiar