Skip to content
This repository was archived by the owner on Jun 3, 2025. It is now read-only.

Final Pull request v2 #17

Merged
JulienHansen merged 12 commits intomainfrom
develop
May 26, 2025
Merged

Final Pull request v2 #17
JulienHansen merged 12 commits intomainfrom
develop

Conversation

@JulienHansen
Copy link
Owner

A bit dumb so i had to redo it, sorry for that.

@JulienHansen JulienHansen self-assigned this May 15, 2025
@JulienHansen
Copy link
Owner Author

The error that occured here is due to the scale down to 0 of the backend, i don't really know why it keeps happening since Monday, i think that clement set the timeout limit to 10s i will change that and everything should go back to normal, in the mean time in the pull request log you can see that everything is working normally

@ThomasVrancken
Copy link
Collaborator

Hi all,

Thanks for the excellent work on this project.
You asked the right design questions, showed the right curiosity and the end application reflects it.
We could see your eagerness in learning new tools.
And we believe you made the right decisions on where to put your focus.

Here are some feedback points for the different sprints:

Sprint 1

++ Great story
++ Motivating why (not just “cure” but rather filter out “benign” disease)
++ Great slides

Sprint 2

++ Good EDA
-- Missed correlation matrix in MS 1 but corrected

Sprint 3

++ Excellent Cloud infrastructure design
++ Clear diagram and design decisions
++ Went above and beyond in configuring your application (image base model questions, automatic deployments…)

Sprint 4

++ Solution uses vertex training
++ Clear separation of the steps
-- It would have saved you some time to use vertex pipeline - but the course explanations came after you actually implemented it

Sprint 5

++ Streamlit
++ Gives confidence interval
++ Monitoring available in streamlit

Sprint 6

++ Full CICD pipeline
++ Pytesting
++ Locust
++ Terraform ideas

Extra feedback

++ Went above and beyond
++ Asked good design questions
++ Only project to actually apply Locust (which led to interesting learnings on throughput caps!)
++ The “future of Deepskin” design was really interesting. You went far into foreseeing your project to a real world application
-- Introduced a few concepts from a standard application that might have not been fully known (QA, waterfall, scrum, …)

You can look back proudly to what you implemented. Clearly you put good energy and creative juice in this project. I suggest showcasing it when applying to jobs, it’s a good vitrine in what you can do (clean codes and nice looking UI).
Hope you had fun and learned interesting things.

The final grade will be uploaded after the exams (project grade included).
Feel free to reach out if you have any questions.

Best regards,
Thomas Vrancken & Matthias Pirlet

@JulienHansen JulienHansen merged commit 419baaf into main May 26, 2025
15 of 16 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants