Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
vidjil
vidjil
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 1,688
    • Issues 1,688
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 84
    • Merge Requests 84
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Incidents
    • Environments
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • CI / CD
    • Repository
    • Value Stream
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • vidjil
  • vidjilvidjil
  • Issues
  • #4339

Closed
Open
Opened Jun 11, 2020 by Mathieu Giraud@magiraudOwner0 of 3 tasks completed0/3 tasks

Bikeshedding du nom : boxes pré-V et post-J, MAX_N_LENGTH_IN_CODE

(Essai avec le bouton magique "resolve all threads in a new issue)

The following discussions from !705 (merged) should be addressed:

  • @mikael-s started a discussion: (+6 comments)

    Pourquoi le MAX_N_LENGTH_IN_CODE ? C'est affiché où ? Si on parse la sortie quel effet ?

  • @mikael-s started a discussion: (+6 comments)

    J'ai des réticences sur ce format type 42/0 sur la sortie : ça casse (encore) la sortie habituelle, ça ne respecte pas notre format .vdj, c'est une information redondante (on a déjà l'information de début du V dans le format .vdj), je ne comprends pas la signification exacte (0 quoi ?), j'ai l'impression que la sémantique est différente des autres /.

  • @magiraud started a discussion: (+2 comments)

    Pour essayer d'avancer (et le sujet de la MR n'était pas là-dessus ;) :

      1. On n'est visiblement d'accord, bref soyons conservateurs, je te propose donc de ne pas mettre ces box au début et à la fin (même si je trouve cela dommage), et à reprendre cette discussion plus tard. Pour l'instant on a peu de séquences comme cela, dans quelque temps on verra plus clair si on en a d'autres, et je pense qu'on a un point d'accord que le name doit décrire des choses biologiques, pas des artefacts, il faudra qu'on ait plus de données.
      1. j'aimerais laisser tout de même le N ... mais si tu insistes on enlève tout : c'est vrai qu'une fois qu'on a enlevé les box au début et à la fin, le N est complètement indépendant de cette MR... j'avais fait cela pour aider le 1), y compris biologiquement, et au final je me suis rendu compte que c'est aussi plus élégant pour lire humainement les sorties (que ce soit stdout ou name) sur quelques exemples).

    Bref, dis-moi ton choix et je détricote en fonction.

To upload designs, you'll need to enable LFS and have admin enable hashed storage. More information
Assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
None
Due date
None
Reference: vidjil/vidjil#4339