Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handling multiple node matching differently in account listing #138

Open
nmathey opened this issue Aug 18, 2023 · 3 comments
Open

Handling multiple node matching differently in account listing #138

nmathey opened this issue Aug 18, 2023 · 3 comments
Labels
bug Something isn't working

Comments

@nmathey
Copy link
Contributor

nmathey commented Aug 18, 2023

Hello,

Lorsque le même nom d'un produit est utilisé dans différentes "comptes" alors le premier seulement est utilisé.
En utilisant le Key ID directement cela semble le faire, mais quid lorsqu'on le veut simplement regrouper un compte sans avoir à "le reconstituer" ligne par ligne.

Exemple:
J'ai sur mon AV gestion libre Linxea Avenir le SP500 UCITS Hedgé ce produit fait également parti de mon AV gestion pilotée WeSave.

AV - WeSave : Lyxor S&P 500 UCITS ETF Daily Hedged Dist id="10341226"
AV - LINXEA AVENIR :  Lyxor S&P 500 UCITS ETF Daily Hedged Dist id="9388332"

Je split mon AV Linxea en deux bucket pour séparer les fonds euros des ETF et dans ce dernier j'utilise bien le Key ID

INXEA_ETF = Bucket(
    "AV Linxea ETF",
    [
        Line("Lyxor S&P 500 UCITS ETF Daily Hedged Dist", key="9388332"),
   ]
)

Pour WeSave je le récupère en enveloppe pour ensuite l'intégrer dans un Folder:

AV_WESAVE = Envelope("WeSave","WESAVE", key="AV - WeSave")

Folder("AV - WeSave", envelope=AV_WESAVE, display=FolderDisplay.LINE),

Sauf erreur de ma part j'ai bien le bon montant sur mon bucket et mon enveloppe mais j'ai toujours le message suivant:

Warning: Line matched with multiple nodes, updating first only: Lyxor source_base.py:119
           S&P 500 UCITS ETF Daily Hedged Dist 

Problème également présent sur mes différent wallet crypto ou j'ai forcément des Ethereum, et autre token/coin identique sur plusieurs comptes différents.

@MadeInPierre
Copy link
Owner

Hey! Je craignais un peu que ça arrive x) Galère à gérer ces lignes qui se ressemblent...

Mais du coup tout fonctionne c'est ça ? Seul le warning reste pour rien ?

Je ne vois pas encore comment résoudre la chose car chaque ligne est indépendante : pour chaque ligne, on parcourt l'arbre entier et collecte toutes Line dans le Portfolio qui correspond aux critères (même key et/ou même envelope). Ensuite, on met ce warning si plusieurs Line qui correspondent sont trouvées et on prend la première par défaut pour mettre le amount venant de Finary/RealT. Soit c'est une erreur de l'utilisateur, soit les critères ont été mal définis...

Je te tiens au courant quand une idée me vient !

@MadeInPierre MadeInPierre added the bug Something isn't working label Aug 18, 2023
@nmathey
Copy link
Contributor Author

nmathey commented Aug 18, 2023 via email

@gierschv
Copy link

Hello,

J'ai eu le même problème lorsque j'ai voulu setup mon portfolio, ça prenait uniquement la première ligne qui correspondait au le nom de la ligne (par exemple un ETF que j'ai sur 2 AV), et du coup le total du portfolio était pas bon, uniquement la 1ere ligne était prise en compte dans le total.

La solution que j'ai eu a été de rajouter les enveloppe sur chaque ligne, ça fonctionne super bien depuis, pas besoin de mettre les ids. 👍

   │   │   │   │   ├── ‣ X € 59% [BNP] Amundi S&P 500 UCITS ETF                                    
   │   │   │   │   ├── ‣ X €  5% [SP2] Amundi S&P 500 UCITS ETF         

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: 🧑‍💻 In Progress
Development

No branches or pull requests

3 participants