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

added the requirments for issue 293 Sprint5 #296

Merged
merged 5 commits into from
Dec 4, 2024

Conversation

Ed0827
Copy link
Collaborator

@Ed0827 Ed0827 commented Nov 25, 2024

Here are the key changes:
Implemented the functionality to control the seven-segment display: It should support showing numbers and characters.
Component defined in YAML file with:
name
bus
device
(I put these changes into under pi4micronaunt-utils/resource/application.yml since Reddy suggested to implement this instead of application.yml under pi4micronaunt)

These two areas met the requirements:
The schematic diagram accurately depicts the wiring for the seven-segment display.
Controller Implementation in the Components folder

Copy link
Collaborator

@yrlmanoharreddy yrlmanoharreddy left a comment

Choose a reason for hiding this comment

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

The approach for implementing the seven-segment display is incorrect. Please revise your approach, make the necessary corrections, and push the updated changes.

@Ed0827
Copy link
Collaborator Author

Ed0827 commented Dec 1, 2024

The approach for implementing the seven-segment display is incorrect. Please revise your approach, make the necessary corrections, and push the updated changes.

Could you please help me how to implement it correctly? I have tried to revise in different ways multiple times, but I still don't know what's wrong with my code. Could you please also specify which implementation was wrong in seven-segment display?

@yrlmanoharreddy yrlmanoharreddy merged commit f0d6c49 into main Dec 4, 2024
6 checks passed
@yrlmanoharreddy yrlmanoharreddy deleted the 7_segement_display_support branch December 4, 2024 02:23
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.

2 participants