TutorTask86_Spring2025_Real_Time_Bitcoin_Price_Analysis_Using_Amazon_EMR_2#252
Merged
tkpratardan merged 14 commits intoJun 12, 2025
Conversation
…rice_Analysis_Using_Amazon_EMR_2
Prahar08modi
previously requested changes
May 14, 2025
Collaborator
Prahar08modi
left a comment
There was a problem hiding this comment.
I would like you to make the following changes:
- Write all your utility/wrapper functions in utility file (for eg., emr_utils.py) like fetch_bitcoin_price, save_record, etc.
- Convert all your .py files to namely 2 jupyter notebooks ( for eg., bitcoin_emr.API.ipynb and bitcoin_emr.example.ipynb).
- The notebooks should only call the functions written in emr_utils.py file.
- Apart from these changes, you should also add markdown files for your project.
Note: Please got through the required files for submission here
Collaborator
Author
|
Hi,
I wanted to let you know that I've completed all the requested updates for
PR #252 (
TutorTask86_Spring2025_Real_Time_Bitcoin_Price_Analysis_Using_Amazon_EMR_2).
The project now fully reflects the real-time pipeline, and includes:
-
Refactored utility functions in bitcoin_emr_utils.py
-
Both Jupyter notebooks: bitcoin_emr.API.ipynb and
bitcoin_emr.example.ipynb
-
Completed markdown files: bitcoin_emr.API.md and bitcoin_emr.example.md
-
Updated README to reflect the real-time architecture
Everything has been tested, and the data is successfully writing to S3.
Please let me know if anything else is needed. I’d appreciate your review
whenever convenient.
Thank you!
Best regards,
Rithika Baskaran
…On Wed, May 14, 2025 at 7:04 PM Prahar Modi ***@***.***> wrote:
***@***.**** requested changes on this pull request.
I would like you to make the following changes:
- Write all your utility/wrapper functions in utility file (for eg.,
emr_utils.py) like fetch_bitcoin_price, save_record, etc.
- Convert all your .py files to namely 2 jupyter notebooks ( for eg.,
bitcoin_emr.API.ipynb and bitcoin_emr.example.ipynb).
- The notebooks should only call the functions written in emr_utils.py
file.
- Apart from these changes, you should also add markdown files for
your project.
*Note*: Please got through the required files for submission here
<https://github.com/causify-ai/tutorials/tree/master/DATA605#submission>
—
Reply to this email directly, view it on GitHub
<#252 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AUAFARKN463BNUA3LA6LGB326PDW3AVCNFSM6AAAAAB3P2PBN6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDQNBRG4YDEMJWGM>
.
You are receiving this because you were assigned.Message ID:
***@***.***>
|
Collaborator
Author
|
This PR includes the final working version of my project: Real-Time Bitcoin Price Analysis Using Amazon EMR. Key Updates:
📁 Final Files:
|
Collaborator
Author
Final Submission: Real-Time Bitcoin Price Analysis Using Amazon EMRThis PR includes the final, tested version of my project including:
All code runs end-to-end in Docker. AWS interactions gracefully fallback when credentials are missing. Ready for grading. |
…rice_Analysis_Using_Amazon_EMR_2
tkpratardan
approved these changes
Jun 12, 2025
…rice_Analysis_Using_Amazon_EMR_2
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR includes the main scripts and notebooks for the project titled "Real-Time Bitcoin Price Analysis Using Amazon EMR", under TutorTask86_Spring2025.
Files added:
bitcoin_ingest.API.py: Pulls real-time Bitcoin price data from an APIbitcoin_processing.example.py: Handles transformation and aggregation of raw price databitcoin_streaming.example.py: Implements real-time processing using PySparkbitcoin_timeseries.example.py: Performs basic time-series analyticsspark_test.py: A lightweight Spark job to validate setuptemplate.API.ipynb: Exploratory notebook with annotated logic and testsPath:
DATA605/Spring2025/projects/TutorTask86_Spring2025_Real_Time_Bitcoin_Price_Analysis_Using_Amazon_EMR/The
README.mdfile will be added in a follow-up commit.