Skip to content

Conversation

@SiddardhReddy-999
Copy link

Created Dashboard,py into the repo
Please merge my pullrequest

Copy link
Author

@SiddardhReddy-999 SiddardhReddy-999 left a comment

Choose a reason for hiding this comment

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

Remove or replace the final Footer word. (critical)

Move or remove np.random.seed(42) inside generate_sample_data so live rows differ. (important)

Insert time.sleep(int(refresh_seconds)) before st.experimental_rerun(). (important)

Change to_csv_bytes to return buffer.getvalue(). (recommended)

Use pd.api.types.is_datetime64_any_dtype(...) for datetime detection. (recommended)

Optional: tweak how default start_time is computed if you want the last time = now. (optional)

Optional: add a warning when requested Y columns don't exist. (UX improvement)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant