ai-cfia / nachet-backend

A flask-based backend for Nachet to handle Azure endpoint and Azure storage API requests from the frontend.
MIT License
1 stars 3 forks source link

issue #51: implement pipeline switching in back end #55

Closed MaxenceGui closed 3 months ago

MaxenceGui commented 5 months ago
MaxenceGui commented 4 months ago

I've added model documentation from the backend. It is still a work in progress. Following this doc, here is a list of what's left to do in the code:

I'm asking for reviews from:

To access the new documentation: docs/nachet-model-documentation.md

rngadam commented 4 months ago

@MaxenceGui New issues were created related to the work here so we can postpone some of the work and get this huge PR merged in main.

Modify the description to add checklist items as completed by referencing postponed tasks to new issues; this gives us a trace between the work that should have been done here and follow up issues.

MaxenceGui commented 4 months ago

@MaxenceGui New issues were created related to the work here so we can postpone some of the work and get this huge PR merged in main.

Modify the description to add checklist items as completed by referencing postponed tasks to new issues; this gives us a trace between the work that should have been done here and follow up issues.

Perfect, will do that. I push my last commit so all checks pass. @RussellJimmies and @rngadam if there is any last-minute review you wanna add, go ahead. If you approve this PR I will rebase and push it to main.

rngadam commented 4 months ago

Seems like the issue links are created manually in the description? I think they look nicer if you use this notation:

* issue #59 
* issue #61 
* issue #60

output:

MaxenceGui commented 4 months ago

Seems like the issue links are created manually in the description? I think they look nicer if you use this notation:

It is I will absolutely change it

rngadam commented 4 months ago

As discussed yesterday, the code here does not seem to be going in the right direction in terms of readability, testability and simplicity

the code is much improved from when I wrote this comment

MaxenceGui commented 3 months ago

@rngadam in case this PR is still far away in your notification

rngadam commented 3 months ago

@RussellJimmies @ibrahim-kabir please revisit, verify fixes and update/resolve conversations as necessary.

@MaxenceGui needs to be rebased, conflicts with main.

MaxenceGui commented 3 months ago

Create issue to follow up with tiny title in Mermaid:

MaxenceGui commented 3 months ago

I understand that there's a follow up PR that depends on this one being checked in first

Yes! I Will correct markdown lint error and rebase before merging into main.

MaxenceGui commented 3 months ago

@RussellJimmies just missing your approval before merging in with main