[PR #332] [CLOSED] [build] update apps_layout.csv from EPSILON_APPS flag in make command #367

New issue

Closed

opened 2026-05-06 13:17:03 +02:00 by BreizhHardware · 0 comments

BreizhHardware commented

2026-05-06 13:17:03 +02:00

Owner

📋 Pull Request Information

Original PR: https://github.com/UpsilonNumworks/Upsilon/pull/332
Author: @AngeDieu
Created: 7/13/2023
Status: ❌ Closed

Base: upsilon-dev ← Head: upsilon-dev

📝 Commits (2)

51d5ed0 [build] update apps_layout.csv from EPSILON_APPS flag in make command
ab854e1 Merge branch 'UpsilonNumworks:upsilon-dev' into upsilon-dev

📊 Changes

2 files changed (+13 additions, -1 deletions)

View changed files

📝 apps/home/Makefile (+2 -1)
📝 apps/home/apps_layout.py (+11 -0)

📄 Description

I did it ! It's my first real modification of Upsilon (and of any c++ project) ! 🎉

I fixed a bug :
When you buid Upsilon changing the app order with the EPSILON_APPS flag, the layout stayed the old one, and the app you clicked on wasn't the shown one.
It's easier to understand with screenshots:

and when I click on it, it opens the python app !

To fix it, the apps_layout.csv is automaticly updated from the EPSILON_APPS flag in the make command.

The Default and HidePython layouts are both updated because of the same bug: the layout and the content of the home app isn't synchronized (pressing the graph icon doen't mean go to the graph icon, but go to the app n°x, which can be different from the graph app).

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/UpsilonNumworks/Upsilon/pull/332 **Author:** [@AngeDieu](https://github.com/AngeDieu) **Created:** 7/13/2023 **Status:** ❌ Closed **Base:** `upsilon-dev` ← **Head:** `upsilon-dev` --- ### 📝 Commits (2) - [`51d5ed0`](https://github.com/UpsilonNumworks/Upsilon/commit/51d5ed04d0c1daf997f7997867d5412f896755a6) [build] update apps_layout.csv from EPSILON_APPS flag in make command - [`ab854e1`](https://github.com/UpsilonNumworks/Upsilon/commit/ab854e16ab3307270cacd6d1864dd2700dc9d74e) Merge branch 'UpsilonNumworks:upsilon-dev' into upsilon-dev ### 📊 Changes **2 files changed** (+13 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `apps/home/Makefile` (+2 -1) 📝 `apps/home/apps_layout.py` (+11 -0) </details> ### 📄 Description I did it ! It's my first real modification of Upsilon (and of any c++ project) ! :tada: I fixed a bug : When you buid Upsilon changing the app order with the EPSILON_APPS flag, the layout stayed the old one, and the app you clicked on wasn't the shown one. It's easier to understand with screenshots: ![image](https://github.com/UpsilonNumworks/Upsilon/assets/106985774/6f5246cc-a033-4b2e-b35e-6f4babe598db) and when I click on it, it opens the python app ! ![image](https://github.com/UpsilonNumworks/Upsilon/assets/106985774/9258246e-b81c-4119-abeb-bbdf2d786d21) To fix it, the apps_layout.csv is automaticly updated from the EPSILON_APPS flag in the make command. The Default and HidePython layouts are both updated because of the same bug: the layout and the content of the home app isn't synchronized (pressing the graph icon doen't mean go to the graph icon, but go to the app n°x, which can be different from the graph app). --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>