entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

widgets: make table widget handle loading back incomplete data (#17061) #960

Merged
fpeters merged 1 commits from wip/17061-table-change into main 2024-01-12 15:11:33 +01:00
Conversation 0 Commits 1 Files Changed 2 +21 -1
fpeters commented 2023-12-23 13:31:09 +01:00
Owner
Copy Link
No description provided.
fpeters added 1 commit 2023-12-23 13:31:10 +01:00
fpeters changed title from WIP: widgets: make table widget handle loading back incomplete data (#17061) to widgets: make table widget handle loading back incomplete data (#17061) 2023-12-26 18:23:03 +01:00
pmarillonnet reviewed 2024-01-03 11:21:10 +01:00
pmarillonnet left a comment
Owner
Copy Link

J’imagine qu’il peut y avoir des cas limites où les colonnes ou lignes ont été modifiés de façon à ce que widget.set_value(value[i][j]) vient être rempli avec une valeur erronée sans pour autant lever une IndexError.
Je me pose la question de faire ce que tu fais dans la PR versus ne pas initialiser de valeur du tout, pour aucune cellule, lorsque la matrice value a des dimensions suspectes, sans savoir ce qui est le mieux.

J’imagine qu’il peut y avoir des cas limites où les colonnes ou lignes ont été modifiés de façon à ce que `widget.set_value(value[i][j])` vient être rempli avec une valeur erronée sans pour autant lever une `IndexError`. Je me pose la question de faire ce que tu fais dans la PR versus ne pas initialiser de valeur du tout, pour aucune cellule, lorsque la matrice `value` a des dimensions suspectes, sans savoir ce qui est le mieux.
tnoel approved these changes 2024-01-05 09:57:19 +01:00
fpeters merged commit c30dbe4893 into main 2024-01-12 15:11:33 +01:00
fpeters deleted branch wip/17061-table-change 2024-01-12 15:11:33 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
tnoel
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#960
No description provided.
Delete Branch "wip/17061-table-change"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?