[M2] <Rolando Asa> — <NBA GOAT Comprehensive Data Analysis>

[ambs78]

Builder Name

Rolando Asa

Cohort

Open Track

GitHub Repo URL

https://github.com/ambs78/dep-data-engineering-open-track-ambs

Commit Hash

77e18a7

Self-Check

• scripts/ingest.py exists and runs without crashing.
• data/raw/ contains at least one real data file (not just .gitkeep).
• requirements.txt lists all dependencies used in ingestion.
• My README documents the data source and how to run the ingestion script.
• The commit hash above points to my latest submission snapshot.

[github-actions]

Deadline notice — M2

⚠️ Submitted after the target deadline. The M2 target deadline was Aug 2, 2026, 11:59:59 PM PHT.

This issue will remain open and continue through the normal automated checks, prerequisite queue, and reviewer process. Continue using this issue for any revisions.

---

DEP submission bot

[github-actions]

Evaluation Results — M2

❌ scripts/ingest.py exists
Add scripts/ingest.py — this is your ingestion script.
✅ data/raw/ contains at least one real data file
✅ requirements.txt exists

Action required: Fix the problems above, push a commit, then comment /recheck <commit-hash> on this issue (7- or 40-character hash both work). Your original submission date remains preserved.

---

DEP submission bot

[ambs78]

I have updated my ingestion script, by renaming it from fetch_nba_goat_stats.py --> ingest.py.
I have updated my commit hash in the ticket to reflect the latest one after renaming the ingest file

[D0R11]

Checklist Criteria

❌ Ingestion script runs end-to-end without errors
❌ Script pulls the M1-confirmed source programmatically
✅ Scraping is compliant (robots.txt + pacing), if applicable (N/A)
❌ Dev environment is replicable (manifest + README steps)
✅ Raw data landed in /data/raw, unmodified and dated

Hi, after checking your latest commit, I can't see the installation steps to run the ingestion script. I do appreciate, however, the processed output that you made, apart from the raw data that was asked of you.

Nonetheless, will mark this as Needs Improvement. Please update your README with the steps. Thank you.

[ambs78]

Hello @D0R11,

Thanks for the feedback!

Would like to clarify on the 'installation steps to run the ingestion script'. Do I need to add this in the README? Since to run the ingestion script, I just double-clicked it, and the code ran.

Also for these items, which parts of the repo should I update? Apologies for these basic questions, I just really have no idea at the moment.
❌ Script pulls the M1-confirmed source programmatically
❌ Dev environment is replicable (manifest + README steps)

Thank you for your patience!

[D0R11]

Hi @ambs78,

Apologies for the late reply. Yes please include that in the README even if adding just 1-2 steps, including installing the modules. Will check the remaining criteria already once I can test the script already. Thank you.

[ambs78]

Thanks @D0R11,

Thanks for your update.
I will get back to you with updates until tomorrow evening (8/27/2026)

[ambs78]

Hi @D0R11,

I've update my README file to include instructions on how to install the modules using requirements.txt and also run the data ingestion using ingest.py.
Please help to review the update, and let me know if I need to add more details.

Thank you!