Skip to content

Level 5 and Level 6: jv-singh#485

Open
jv-singh wants to merge 11 commits into
Life-Atlas:masterfrom
jv-singh:master
Open

Level 5 and Level 6: jv-singh#485
jv-singh wants to merge 11 commits into
Life-Atlas:masterfrom
jv-singh:master

Conversation

@jv-singh
Copy link
Copy Markdown
Contributor

Hi team,

Here is my complete submission for Level 5 and Level 6.

Live Streamlit Dashboard: https://lpi-developer-kit-6ljqxbc89s5efpisjg78lp.streamlit.app

Level 6 Checklist Completed:
✅ Neo4j database seeded successfully.
✅ Streamlit dashboard deployed successfully on Streamlit Community Cloud.
✅ All 4 required pages are functional and pulling live graph data.
✅ Self-Test page is passing with a perfect 20/20 score.

Folder Structure

Here is how my submission is organized:

submissions/
└── jv-singh/
    ├── level5/
    │   ├── answers.md
    │   └── schema.md
    └── level6/
        ├── .streamlit/
        ├── .venv/
        ├── data/
        ├── .env
        ├── .env.example
        ├── .gitignore
        ├── app.py
        ├── DASHBOARD_URL.txt
        ├── README.md
        ├── requirements.txt
        └── seed_graph.py

Copy link
Copy Markdown
Contributor

@LifeAtlas LifeAtlas left a comment

Choose a reason for hiding this comment

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

L5 Score: 95/100 | L6 Score: 88/100 ✅ 🏆

TOP SUBMISSION. L5: By far the most detailed and thoughtful answers (1031 lines). Identifies data-quality discrepancies (14 workers not 13, Per Hansen not Per Gustafsson). Parameterized Cypher. L6: All pages + bottleneck alerts, MERGE + constraints, SPOF flagging, 6-check self-test. Outstanding work.

Copy link
Copy Markdown
Contributor

@LifeAtlas LifeAtlas left a comment

Choose a reason for hiding this comment

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

CORRECTION: Security flag added

L5: 95/100 (unchanged) | L6: 68/100 (was 88, -20 for credentials)

Your L5 is still the best in the cohort. However, your L6 committed .env AND .streamlit/secrets.toml with real Neo4j credentials (URI + password in plaintext). Per rubric: -20, mandatory fix before merge.

Action required: Remove .env and .streamlit/secrets.toml from the PR. Add them to .gitignore. Keep .env.example only.

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