Ian Letourneau letian
  • Joined on 2025-04-08
letian pushed to feat/applicationModule at NationTech/harmony 2025-07-01 18:42:49 +00:00
b621a92c82 remove unused go application score
letian pushed to feat/applicationModule at NationTech/harmony 2025-07-01 15:51:26 +00:00
2c90d50168 format code
letian pushed to feat/applicationModule at NationTech/harmony 2025-07-01 15:34:48 +00:00
c02763a6fc Adjust imports
letian commented on pull request NationTech/harmony#70 2025-07-01 14:12:47 +00:00
feat: Application module architecture and placeholder features

naming could be revisited, but good enough for now

letian commented on pull request NationTech/harmony#70 2025-07-01 14:12:33 +00:00
feat: Application module architecture and placeholder features

is_installed and uninstall seem unnecessary for now

letian commented on pull request NationTech/harmony#25 2025-04-29 21:51:28 +00:00
WIP: Add support for profiles

Je n'ai pas le temps de le faire maintenant, mais pour regler le probleme de temporal coupling que cette methode introduit, il faudrait simplement a la supprimer et reutiliser create_interpret pour y mettre la logique de merge la config avec le profile avant de creer l'interpret.

letian commented on pull request NationTech/harmony#25 2025-04-29 21:47:53 +00:00
WIP: Add support for profiles

Dans ce cas, ca semble toucher 2 aspects differents en fait:

  1. Le developpeur final devrait pouvoir avoir le controle de configurer certains aspects d'un score en fonction du profil (ce que cette…
letian commented on pull request NationTech/harmony#25 2025-04-29 20:39:01 +00:00
WIP: Add support for profiles

Without mentioning it at all, Gemini have the same concerns I have 😆

     

Weaknesses and Concerns:

* Error Handling: The panic!("Not good") in LAMPScore::apply_profile is completely…
letian pushed to runtime-profiles at NationTech/harmony 2025-04-29 20:31:01 +00:00
letian pushed to runtime-profiles at NationTech/harmony 2025-04-29 20:16:07 +00:00
letian commented on pull request NationTech/harmony#25 2025-04-29 20:08:52 +00:00
WIP: Add support for profiles

I'm not a big fan of this because this introduces a temporal coupling. We have to remember to call it before creating our interpret otherwise we're screwed.

letian pushed to runtime-profiles at NationTech/harmony 2025-04-29 20:07:50 +00:00
letian created pull request NationTech/harmony#25 2025-04-29 20:01:36 +00:00
WIP: Add support for profiles
letian pushed to runtime-profiles at NationTech/harmony 2025-04-29 19:56:22 +00:00
letian created branch runtime-profiles in NationTech/harmony 2025-04-29 19:56:22 +00:00