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.

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

Add Streamlit app for real-time IoT heart rate and stress monitoring #194

Merged

Conversation

Shibika-Roy
Copy link
Contributor

“Add Streamlit app for real-time IoT heart rate and stress monitoring
This dashboard enables real-time monitoring of heart rate and stress levels to personalize user relaxation experiences.

Simulated Data Monitoring:

Observe the heart rate and stress level metrics displayed in real time. These values update every second to simulate continuous data monitoring.
Confirm that both metrics (heart rate in BPM and stress level on a 1–10 scale) are displayed and updating smoothly.
User Experience:

The interface should provide a simple, easy-to-read layout with real-time updates on physiological data.
This simulation sets the stage for integrating actual IoT data for heart rate and stress monitoring.
This addition enables real-time physiological data monitoring, a feature that supports the app's mission to provide personalized and effective relaxation guidance. Future updates could replace simulated data with actual IoT device readings for further enhancement.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

🎉 Thank you for your contribution! Your pull request has been submitted successfully. A maintainer will review it as soon as possible. We appreciate your support in making this project better.

Please follow these guidelines for smooth collaboration:

  1. After completing a task, add your name and details to the Contributors Table/List.
  2. Attach relevant screenshots or videos of your contribution for transparency.
  3. Tasks are assigned on a first-come, first-served basis.
    • Avoid spamming requests for already-assigned tasks.
    • Respect the queue and wait for new tasks.
  4. Propose New Ideas: Discuss new features or suggestions in the "Ideas" channel before starting work.

@Shibika-Roy
Copy link
Contributor Author

please level as gssoc-extd hacktoberfest accepted

@zoobiasyed
Copy link
Collaborator

@Amna-Hassan04 look into this, with only one day left, kindly assign this task too 1 or 2 contributors. The idea Shibika has provided will add more power to the app. If you think we have time than we should add this to the app.

@Amna-Hassan04
Copy link
Owner

@Shibika-Roy add all the necessary screenshots / video showcasing yoru work, also add the issue that you worked on

@Shibika-Roy
Copy link
Contributor Author

VID-20241029-WA0004.mp4

@Shibika-Roy
Copy link
Contributor Author

Shibika-Roy commented Oct 29, 2024

This issue you had been already assigned to me 2 weeks ago
it was labled also as level 3

@Shibika-Roy
Copy link
Contributor Author

If this app comes with hardware devices in future it will work in this way

@Amna-Hassan04 Amna-Hassan04 self-requested a review October 30, 2024 05:48
@Amna-Hassan04
Copy link
Owner

@Shibika-Roy plz add this as a tab in teh original app. plus with the heading you can add beta. add the relevant way any user can connect and check their heart rate and stress levels. plus mention the issue number you worked on.

@Shibika-Roy
Copy link
Contributor Author

Shibika-Roy commented Oct 30, 2024

My issue was #77

@zoobiasyed
Copy link
Collaborator

@Amna-Hassan04 kindly review this.

@Shibika-Roy
Copy link
Contributor Author

Shibika-Roy commented Oct 30, 2024

what would be the tab name??
...would it be like 'monitoring' tab can you please suggest me ...

@Shibika-Roy
Copy link
Contributor Author

I’ve added the Monitoring (Beta) tab to the Streamlit app, which provides real-time heart rate and stress level monitoring and includes connection instructions for IoT-enabled devices. This feature allows users to monitor their metrics directly in the SereniFi Guide app.

Conflict Status: I attempted to resolve conflicts between my branch and the main branch, but some conflicts remain that I couldn't fully resolve. If possible, could you assist with resolving the remaining conflicts to ensure this feature integrates smoothly?
Feature Details:

Tab: Monitoring (Beta)
Functionality: Real-time metrics with simulated data for heart rate and stress, along with connection instructions.
Instructions for Use: Users can follow setup instructions to connect their IoT devices and view their metrics in real time.

please inform me...

@zoobiasyed
Copy link
Collaborator

@Amna-Hassan04 kindly look into this, the conflicts are still not fully resolved.

Copy link
Owner

@Amna-Hassan04 Amna-Hassan04 left a comment

Choose a reason for hiding this comment

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

first of all dont write your name in comment in the code, your profile will automatically show in the contributers column in the repository, While adding the tab you have changed the names of teh other tabs, u are also using a old version of the project which needs to be updated asap. now if u have added a new tab then plz make a video showcasing it and how it works overall with the app, also it would be better to show the process for connecting a monitoring device so I am sure to merge your pr. their are only 2 days left so for timely merge of your pr plz complete these steps asap so I can merge your pr.

@Shibika-Roy
Copy link
Contributor Author

Shibika-Roy commented Nov 8, 2024

ok ok ... I am new on this I realy don't know

@Shibika-Roy
Copy link
Contributor Author

VID-20241108-WA0006.mp4

Copy link
Contributor Author

@Shibika-Roy Shibika-Roy left a comment

Choose a reason for hiding this comment

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

ok

@Shibika-Roy
Copy link
Contributor Author

Video_2024_11_08_23_04_46.mp4

@zoobiasyed
Copy link
Collaborator

@Shibika-Roy did you complete all the tasks Amna told you to improve or change? If yes than provide with the relevant pictures.

@Shibika-Roy
Copy link
Contributor Author

@Shibika-Roy did you complete all the tasks Amna told you to improve or change? If yes than provide with the relevant pictures.

yes

@zoobiasyed
Copy link
Collaborator

@Amna-Hassan04 kindly look into this.

Copy link
Owner

@Amna-Hassan04 Amna-Hassan04 left a comment

Choose a reason for hiding this comment

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

Shibika I asked u to make a separate tab that shoudl show in the navbar, it really doenot look good how u have implemented this, i do not want you lose poinst by not getting your pr merged, for now just do this that add you montioring app functionality at the end of the calm space tab and remove this side bar. u still have almost 2 hours plz complete this asap so i can merge this.

@Shibika-Roy
Copy link
Contributor Author

ok

@Amna-Hassan04
Copy link
Owner

@Shibika-Roy Good Luck, Im waiting for u fix this so u dont lose any points. the time is only till 7 pm today.

@Shibika-Roy
Copy link
Contributor Author

okk I understand @Amna-Hassan04

@Amna-Hassan04
Copy link
Owner

@Shibika-Roy and be sure to use the latest version of the project

@Amna-Hassan04
Copy link
Owner

@Shibika-Roy also add a screenshot of how it looks

@Amna-Hassan04 Amna-Hassan04 merged commit 2dedb49 into Amna-Hassan04:main Nov 10, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants