GraphiteEditor / GraphiteEditor/Graphite
Move network import type from NodeInput::Network to NodeNetwork in order to enable automatic Into node insertion
- Langage dominant
- Rust
- Étoiles
- 27.2k
- Forks
- 1.3k
- Merge moyen
- 20 h 5 min
- PR mergées (30 j)
- 57
Description
Instead of annotating type for each NodeInput::network usage, the types should be annotated in a single vec for every import.
For example we do something like
```
pub fn node(input1){
subnode(input1: String)
subnode(input1: String)
etc
}
```
rather than
```
pub fn node(input1: String){
subnode(input1)
subnode(input1)
etc
}
```
Technically I dont think we even need to annotate the parameters, since their types can be resolved at compile time. However, this system would be useful/necessary for automatic into node insertion (#1890, which can also be used to solve #1919), as well as displaying what types can be connected to a node when hovering over the input.
Discussions here:
https://discord.com/channels/731730685944922173/731738914812854303/1253116397370998794
https://discord.com/channels/731730685944922173/860830913390575646/1271586374314885201
https://discord.com/channels/731730685944922173/860830913390575646/1271605307771981987
And a more recent recap asking for a reminder about what this is and its status: https://discord.com/channels/731730685944922173/1059075292414619689/1387941282701119550
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Piste de recherche
Commencez par suivre NodeInput::Network et NodeNetwork, puis examinez les discussions liées et les issues associées #1890 et #1919 afin d’établir la représentation de type prévue. C’est terminé lorsque les imports utilisent un vecteur unique annoté avec un type, qui prend en charge l’insertion automatique de nœuds Into et expose les types connectables pour les informations affichées au survol.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- rust
- Domaine
- compilers
- Type d'issue
- Fonctionnalité
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 25/100