Power estimate: use the imported binary of a version as its base - #18
Power estimate: use the imported binary of a version as its base#18COSSART-FR wants to merge 1 commit into
Conversation
The power-estimate modal rebuilt a non-live version as Ori + persisted map edits. A version created by importing a file only writes version-<id>.bin and keeps its modifications in memory until the user saves, so estimating it from the dashboard (or with another version open in the editor) ran on the stock file and showed the stock figures. Start from readVersionBinary(versionId) when it exists (same precedence as buildVersionFileData in the editor), fall back to the original otherwise. The original still anchors the injector-flow model.
|
Salut Mathis, et d'abord désolé. J'ai poussé la 1.1.8 hier et j'ai oublié d'intégrer ton commit. Le correctif, lui, est bien dans la version, et c'est le tien : j'ai comparé ta branche avec ce qui est parti, la logique est identique ligne pour ligne, seul le commentaire diffère. Mais ton commit n'est pas dans l'historique, et ça c'est de ma faute, pas celle du fonctionnement du dépôt. Ce qui s'est passé : master ne garde qu'un seul commit par version, tout est écrasé dans un « Release X.Y.Z ». Au lieu de faire suivre ta paternité, j'ai réécrit la modification dans ce commit. Ça n'aurait pas dû se passer comme ça et je m'en veux. Tu es crédité là où je pouvais encore le faire : dans les notes de version de la 1.1.8, et dans une nouvelle rubrique Contributeurs que je viens d'écrire pour le README, qui sera en ligne avec la prochaine version. Ton nom est en haut, dans la partie code. Pour la prochaine, je fais les choses correctement. Le commit de release portera une ligne Co-authored-by avec ton nom et l'adresse de ton commit, comme ça GitHub t'affiche comme auteur et te compte dans les contributeurs du dépôt, squash ou pas. Une chose que je voudrais te demander, si ça ne te dérange pas : est-ce que tu peux passer tes deux pull requests en brouillon, celle-ci et celle de l'EDC16CP31 ? Comme ça elles n'ont plus l'air d'attendre une relecture qui ne vient pas, et elles restent l'endroit où le travail atterrira le moment venu. Rien n'est fermé de mon côté, surtout la Mercedes, il lui manque juste le corpus dont on a parlé. Merci encore pour les deux, et désolé encore pour l'histoire du crédit. |
The power-estimate modal rebuilt a non-live version as Ori + persisted map edits. A version created by importing a file only writes version-.bin and keeps its modifications in memory until the user saves, so estimating it from the dashboard (or with another version open in the editor) ran on the stock file and showed the stock figures.
Start from readVersionBinary(versionId) when it exists (same precedence as buildVersionFileData in the editor), fall back to the original otherwise. The original still anchors the injector-flow model.