TI-Nspire Dev Day - Page 4

Pour le problème du ROM dump 84+:
http://svn.calcforge.org/viewvc/calcforgelp?view=revision&revision=4581
http://svn.calcforge.org/viewvc/calcforgelp?view=revision&revision=4582

ExtendeD (./90) :
Tu ne compares pas ce qui est comparable, et tu n'as pas la moindre idée de ce qui s'est passé durant ces 2 mois. Pourquoi se vanter ainsi ?

Tu ne comprends pas, je ne me vante pas du tout! Ce que je veux dire, c'est qu'il y aurait eu plein de temps en ces 2 mois d'implémenter ces fonctionnalités dans les libs CalcForge, il aurait été suffisant de me demander les fonctions qu'il faut, ou de les demander à Lionel, ou de les implémenter toi-même, dans aucun des cas, ça n'aurait retardé la sortie de manière significative (voire du tout).

Mouai bref...

le but était de développer ndless, les outils autour s'améliorent dans un second temps.

...faites vous plein de calins et de bisous !

L'important c'est que NDless est la, qui n'a pas fait quoi, on s'en cogne je pense...tout le monde à abattu un travail hors du commun en tout cas...

Vision simpliste et idéaliste, et certains se sont suffisamment pris de beignes pour ne pas se faire réavoir par un certain esprit bisounours

Voilà, tout à fait.

Et puis je crois que personne n'a envie de voir Kevin et Folco se faire des câlins et des bisoux (au sens propre en tout cas), à commencer par les deux intéressés

Et à continuer par ma femme

Et voilà la dernière pièce manquante:
http://svn.calcforge.org/viewvc/calcforgelp?view=revision&revision=4590

Add a calcprotocols_calc_nspire_get_attributes function giving access to this command of the TI-Nspire and TI-Nspire CAS (but it may get replaced by higher-level APIs in the future).

Retourne 402 si le dossier n'existe pas, 0 si le dossier existe, autre chose s'il y a une autre erreur.

Bref, tout devrait être prêt côté libs CalcForge pour un portage de Ndless.

Dis-donc, Kevin, r4588:4589 constitue la deuxième réimplémentation indépendante de code que j'ai déjà posté sur #ti (channel IRC publiquement accessible avec des logs publics, dont tu connais l'existence), en une semaine. Code qui n'est, pour la même raison que la première fois (tests insuffisants) pas committé - mais au moins, il est publiquement accessible.

Faire une erreur une fois, par le fait de ne pas fréquenter les endroits de la communauté où il se passe quelque chose (non, #tigcc n'en est pas un, pas plus que #GCC4TI d'ailleurs), c'est une erreur. Je l'ai déjà signalée en ./72.
La faire une deuxième fois, une semaine après la première, parce que tu n'as toujours pas décidé de te tenir au courant, c'est une faute.
Certes, tu n'es pas spécialement le bienvenu sur #ti si tu y importes ton comportement habituel qu'on connaît tous dans la communauté TI-*, et que la communauté Fedora est en train de comprendre - mais au moins, lis les logs, puisqu'il y en a !

Ca se voit tout de suite que r4588:4589 est une réimplémentation indépendante et qu'elle n'est pas testée (pas plus qu'au moment de leur commit, tes nouvelles fonctions spéciales Nspire non accessibles de l'UI, et pas plus non plus que le commit qui introduisait l'utilisation de cmd_execute pour lancer le ROM dump sur 84+ et que j'ai debuggé pour toi, voir ./88). En effet, r4588:4589 comporte (au moins) deux bugs qui sont chacun suffisants pour empêcher le dump de fonctionner correctement.
Ces deux bugs sont évidemment absents de mon implémentation, parce que moi, je les ai rencontrés et corrigés quand j'ai testé...

Ah, et si tu tiens à faire porter la responsabilité du fait que tu n'aies pas été au courant à quelqu'un d'autre que toi... tu as le choix: Tyler, cemeyer, BrandonW, sir_lewk étaient tous connectés sur #ti au moment où j'ai posté

ça me fait mal au coeur de lire ce genre de message. je comprends pourquoi il est la mais est il utile?

grand bien lui fasse si il a voulu réimplémenter. Mettez vous d'accord pour avoir un truc qui marche quel que soit le svn utilisé, mais steuplay, quand Kevin attaque pas, ne t'y mets pas à ton tour Si il a négligé ton code, et que tu veux qu'il l'intègre, propose le lui une fois de plus!

faites un bisou merde! le but est de se battre sur qui a fait quoi, ou d'intégrer ces libs dans ndless?

(+1, je voulais pas me mêler dans vos discussions puisque je n'y participe pas, mais de vue extérieure j'ai la même sensation )

Lionel Debroux (./102) :
Dis-donc, Kevin, [...]

Et le rapport de ce rant avec la Nspire? Je n'étais pas au courant de ton patch, c'est normal, il n'est une fois de plus pas committé. J'ai autre chose à faire aussi que d'aller courir après tes patches (surtout dans un log d'un chan IRC complètement indépendant de la LPG). J'ai discuté du protocole avec Brandon, je l'ai implémenté aussitôt que possible, sachant que je n'ai pas que ça à faire, et que je n'ai aucun moyen de tester le code.

Les 2 bogues devraient être corrigés:
http://svn.calcforge.org/viewvc/calcforgelp?view=revision&revision=4591
http://svn.calcforge.org/viewvc/calcforgelp?view=revision&revision=4593


Mais pour revenir au sujet, qui, je le rappelle, est la Nspire:

et qu'elle n'est pas testée (pas plus qu'au moment de leur commit, tes nouvelles fonctions

Comment veux-tu que je teste? Je n'ai pas de Nspire! Je committe ce que j'ai, histoire qu'on ne s'amuse pas à réinventer la roue, à la communauté de tester. J'ai implémenté à la lettre le protocole documenté sur Hackspire qui a déjà servi pour Nspire8x, donc il y a des chances que ça marche du premier coup.

spéciales Nspire

Il n'y a qu'une fonction spéciale Nspire (calcprotocols_calc_nspire_get_attributes). La fonction calcprotocols_calc_copy_file est implémentée pour tous les modèles, elle copie à travers le PC pour les modèles autres que la Nspire tout simplement. (Pour la Nspire, la copie native est utilisée.)

non accessibles de l'UI

Ces fonctions ont été rajoutées exprès pour permettre un portage de Ndless, donc forcément je ne vois pas pourquoi elles devraient être obligatoirement présentes dans CalcForgeLP. Surtout calcprotocols_calc_nspire_get_attributes n'a vraiment strictement rien à faire dans l'interface utilisateurs telle qu'elle se présente actuellement (le dirlist récupère déjà les informations de taille et de type, et pour la date, on ne sait même pas exactement ce que le fameux uint32_t veut dire concrètement).

quand Kevin attaque pas

Du moins, il ne t'attaque pas toi, et il n'attaque pas à un endroit que tu lis

propose le lui une fois de plus!

As-tu remarqué qu'entre autres, ./102 indique où il peut trouver le code ?

grand bien lui fasse si il a voulu réimplémenter.le but est de se battre ou d'intégrer ces libs dans ndless?

Le but général est de proposer des features et bugfixes aux utilisateurs. Et perdre du temps en duplication d'efforts, parce que Kevin ne se tient pas au courant, n'est pas la façon la plus efficace d'y arriver, hmm ?

je n'ai aucun moyen de tester le code.Comment veux-tu que je teste? Je n'ai pas de Nspire!

Tu as tort, c'est exactement le contraire de ce qu'il faut faire pour améliorer la qualité des logiciels LPG, dont tu te plains souvent bien que tu en sois responsable en tant que contributeur. Je me suis procuré quatre machines, pour pouvoir faire des tests.

spéciales Nspire

Il n'y a qu'une fonction spéciale Nspire

Je voulais parler de calcprotocols_calc_new_fld et des autres fonctions bas niveau, il y en a plusieurs.

J'ai implémenté à la lettre le protocole documenté sur Hackspire qui a déjà servi pour Nspire8x, donc il y a des chances que ça marche du premier coup.

A condition que la doc soit correcte: sache que j'y ai récemment corrigé une erreur.

Lionel Debroux (./106) :

propose le lui une fois de plus!

As-tu remarqué qu'entre autres, ./102 indique où il peut trouver le code ?

As-tu remarqué que ./102 ne contient rien qui ressemble à un lien et qu'il m'a fallu faire un boulot de détective pour trouver ton fameux lien?

Au fait, quid de ma question posée sur #tigcc (et dont j'autorise évidemment la publication par moi-même )?

<Kevin_Kofler> Lionel_Debroux: Is there a concrete reason why you run cmd_s_execute with EID_PRGM rather than EID_ASM in dump_rom_2?<Kevin_Kofler> BrandonW says it shouldn't matter, EID_ASM is more correct, and if it doesn't work, we need to fix the execute command as well, because that also uses EID_ASM for ASM programs.

(Oui, c'est à ce genre de communication technique que sert avant tout #tigcc…)

grand bien lui fasse si il a voulu réimplémenter.le but est de se battre ou d'intégrer ces libs dans ndless?

Le but général est de proposer des features et bugfixes aux utilisateurs. Et perdre du temps en duplication d'efforts, parce que Kevin ne se tient pas au courant, n'est pas la façon la plus efficace d'y arriver, hmm ?

Parce que tu ne me mets pas au courant, plutôt. Et aussi parce que tu ne sais pas utiliser un SCM.

je n'ai aucun moyen de tester le code.Comment veux-tu que je teste? Je n'ai pas de Nspire!

Tu as tort, c'est exactement le contraire de ce qu'il faut faire pour améliorer la qualité des logiciels LPG, dont tu te plains souvent bien que tu en sois responsable en tant que contributeur. Je me suis procuré quatre machines, pour pouvoir faire des tests.

Je ne vais pas débourser dans les 1000 euros pour acheter n calculatrices (au fait, j'en ai déjà 2 (TI-89, TI-92+), plus une TI-85 que ma mère utilise, mais que je peux réemprunter au cas où, ce n'est pas vrai que je n'ai qu'une seule calculatrice comme tu as dit sur #ti) pour un projet entièrement bénévole!

arrêtez de vous attaquer réciproquement ! y'a du passif OK, mais bon ça va quoi la guérilla!

je pense que trouver quelqu'un qui a une nspire pour tester les commits est faisable, non?

As-tu remarqué que ./102 ne contient rien qui ressemble à un lien et qu'il m'a fallu faire un boulot de détective pour trouver ton fameux lien?

Le lien des logs de #ti a été posté dans un autre topic de cette même section de yAronet !

<Kevin_Kofler> Lionel_Debroux: Is there a concrete reason why you run cmd_s_execute with EID_PRGM rather than EID_ASM in dump_rom_2?<Kevin_Kofler> BrandonW says it shouldn't matter, EID_ASM is more correct, and if it doesn't work, we need to fix the execute command as well, because that also uses EID_ASM for ASM programs.

Je ne me connecte à IRC depuis le boulot qu'exceptionnellement, je n'avais donc pas vu ces lignes.
J'ai utilisé EID_PRGM puisqu'il n'y a (d'après Brandon) pas de différence entre les deux, et ça fonctionne. Je ne suis pas _sûr_ d'avoir testé EID_ASM, cependant. Proceed at your own risk.

Et aussi parce que tu ne sais pas utiliser un SCM.

Du moins, j'ai une façon différente de la tienne. Déjà débattu, on ne va pas y revenir

Je ne vais pas débourser dans les 1000 euros pour acheter n calculatrices [...] pour un projet entièrement bénévole!

Tu as le droit de faire ce choix (même s'il est basé sur des données fausses - ça coûte moins de 1000 euros de se monter un jeu de machines très représentatif des machines actuelles, même en les achetant neuves) - tout comme j'ai le droit d'avoir fait le choix contraire

(au fait, j'en ai déjà 2 (TI-89, TI-92+), plus une TI-85 que ma mère utilise, mais que je peux réemprunter au cas où, ce n'est pas vrai que je n'ai qu'une seule calculatrice comme tu as dit sur #ti)

ACK. Mais tes TI-68k sont deux calculatrices du même type ancien (LIO seul), et l'autre est une calculatrice d'une famille obsolète. Ca n'est donc pas avec ce jeu de machines que tu vas pouvoir faire des tests significatifs.

squalyl (./108) :
je pense que trouver quelqu'un qui a une nspire pour tester les commits est faisable, non?

La meilleure manière de tester ces fonctions est de porter Ndless et de tester le résultat. Idéalement effectué par quelqu'un qui possède une Nspire. J'ai fait mon boulot de mainteneur des libs CalcForge (en attendant les bug reports).

Et au fait, ExtendeD et geogeo, pourrait-on s'il vous plaît avoir la fameuse licence duale (MPL/GPL plutôt que juste MPL) discutée plus haut (et pour laquelle au moins ExtendeD semblait partant), histoire de pouvoir faire une version utilisant les libs libres sans problèmes légaux? Parce que le conflit de licences est le principal obstacle restant maintenant.