P4Pillon/Krys4lide
4
2
Fork 0
Code Issues 27 Pull Requests 3 Actions Packages Projects 1 Releases Activity

Implémentation du parsing complet des données issues de SSV_LireConfig parsing, en utilisant deku pour une approche déclarative des règles de parsing binaire #75

Open
florian_briand wants to merge 1 commits from feat/38-fsv-lire-config-parsing into main
pull from: feat/38-fsv-lire-config-parsing
merge into: P4Pillon:main
Conversation 0 Commits 1 Files Changed 9 +824 -30
florian_briand commented 2024-10-03 23:32:28 +02:00
Owner
Copy Link

Détails

Largement inspiré du code de la #69 développé par @theo dans le cadre du ticket #38

  • Implémentation d'un usage de deku, une librairie qui permet de configurer des parseurs de binaire dans un style déclaratif
    • Usage de deku sur l'ensemble des données issues de la fonction SSV_LireConfig
    • Implémentation de nombreux tests unitaires

Contribute to #38

Pourquoi ?

No response

Documentation

No response

### Détails Largement inspiré du code de la #69 développé par @theo dans le cadre du ticket #38 - Implémentation d'un usage de [deku](https://docs.rs/deku/latest/deku), une librairie qui permet de configurer des parseurs de binaire dans un style déclaratif - Usage de deku sur l'ensemble des données issues de la fonction SSV_LireConfig - Implémentation de nombreux tests unitaires Contribute to #38 ### Pourquoi ? _No response_ ### Documentation _No response_
florian_briand added the
enhancement
module/sesam-vitale
 labels 2024-10-03 23:32:28 +02:00
florian_briand self-assigned this 2024-10-03 23:32:28 +02:00
florian_briand added 1 commit 2024-10-03 23:32:28 +02:00 
Co-authored-by: theo <t.lettermann@criteo.com>
florian_briand added this to the 0 - POC project 2024-10-03 23:32:28 +02:00
florian_briand added spent time 2024-10-03 23:35:54 +02:00
18 hours 52 minutes
florian_briand requested review from kosssi 2024-10-03 23:36:49 +02:00
florian_briand requested review from theo 2024-10-03 23:36:49 +02:00
florian_briand reviewed 2024-10-03 23:42:44 +02:00
crates/fsv/src/fsv_parsing/groups/ssv_lire_config.rs Outdated
@ -0,0 +60,4 @@
pub software_version: ReaderSoftwareVersion, // 4 CA
pub reader_datetime: ReaderSoftwareDate, // 12 CN
pub software_checksum: AlphaNumericString, // 4 CA
}
florian_briand commented 2024-10-03 23:42:44 +02:00
Author
Owner
Copy Link

Le choix d'implémentation fait ici, sans créer de type spécifique à chaque champ, est en débat dans le ticket #74

  • Faire un choix d'implémentation et unifier le style en accord avec ce choix
Le choix d'implémentation fait ici, sans créer de type spécifique à chaque champ, est en débat dans le ticket #74 - [ ] Faire un choix d'implémentation et unifier le style en accord avec ce choix
florian_briand force-pushed feat/38-fsv-lire-config-parsing from c62f10f0aa to 4e3387cef9 2024-10-04 22:17:07 +02:00 Compare
florian_briand changed target branch from 38-fsv-high-level-lib to main 2024-10-09 22:47:52 +02:00
florian_briand force-pushed feat/38-fsv-lire-config-parsing from 4e3387cef9 to 8f935ab81e 2024-10-09 22:50:47 +02:00 Compare
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feat/38-fsv-lire-config-parsing:feat/38-fsv-lire-config-parsing
git checkout feat/38-fsv-lire-config-parsing
Sign in to join this conversation.
Reviewers
No reviewers
kosssi
theo
Labels
Clear labels   
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
independant

Tickets réalisables indépendaments des travaux prévus dans la roadmap - très approprié pour des contributions ponctuelles

  
invalid

Something is wrong

  
module/autre

Pour toute autre partie du produit non lié aux autres étiquettes disponibles

  
module/backend

Serveur backend, opérant les accès données et système

  
module/desktop

Encapsulation de la webview dans Tauri

  
module/docs

Documentation (docs)

  
module/frontend

Interfaces utilisateur⋅ices

  
module/scripts

Scripts utiles au setup et/ou au build (scripts)

  
module/sesam-vitale

Moteur SESAM-Vitale

  
module/utils

Librairie de fonctions utilitaires

  
open-source

Les tâches contribuant à l'ouverture et au partage du code

  
question

More information is needed

  
to-triage

À trier

  
wontfix

This won't be fixed

No Label
bug
duplicate
enhancement
help wanted
independant
invalid
module/autre
module/backend
module/desktop
module/docs
module/frontend
module/scripts
module/sesam-vitale
module/utils
open-source
question
to-triage
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
0 - POC
Assignees
Clear assignees
No Assignees
florian_briand
1 Participants
Notifications
Total Time Spent: 18 hours 52 minutes
florian_briand
18 hours 52 minutes
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: P4Pillon/Krys4lide#75
No description provided.
Delete Branch "feat/38-fsv-lire-config-parsing"

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?