Réunion du 02-11-2021
Bientôt la traduction
Introduction
[11:03] Andrew Hellershanks : Hm... le cube de journalisation ne fonctionne pas aujourd'hui. [11:03] Andrew Hellershanks : Bonjour à tous. [11:03] Ubit Umarov : opos di di kill it ? [11:03] Andrew Hellershanks : Ubit, je ne sais pas. Qu'est-ce que tu lui as fait ? [11:03] Gavin.Hird @grid.xmir.org:8002 : Pauvre chose, hehe. [11:04] Ubit Umarov : étrange que la notecard soit vide. [11:04] Andrew Hellershanks : Le script qu'elle contient s'appelle juste "New script" ? ?? [11:04] Ubit Umarov : bien sûr [11:05] Andrew Hellershanks : :) [11:05] Ubit Umarov : y a-t-il d'autres noms pour les scripts ? [11:06] Ubit Umarov : oups, il semble que je l'ai fait. [11:06] Andrew Hellershanks : Pas beaucoup de nouvelles à rapporter en ce qui concerne les changements d'OpenSim cette dernière semaine. [11:06] Ubit Umarov : un problème avec les timers grrr. [11:07] Gavin.Hird @grid.xmir.org:8002 : Avez-vous résolu le problème des timers ? [11:07] Andrew Hellershanks : Problèmes de minuterie ? [11:07] Ubit Umarov : juste un peu modifié. [11:07] Ubit Umarov : je suppose que vous devez faire le log de la manière forte cette fois-ci :( [11:08] Andrew Hellershanks : Ce n'est pas un problème tant que la région ne se plante pas. Même si c'est le cas, j'enregistrerai le journal de discussion avant de fermer mon viewer. [11:08] Ubit Umarov: ok [11:09] Ubit Umarov : la machine à logs est aussi morte. [11:09] Ubit Umarov : hmm ou pas
Mantis et tests
[11:10] Vincent.Sylvester @hg.zetaworlds.com:8002 : J'ai réussi à résoudre 370 d'issues sur la mantis en tout, dont une bonne centaine au cours des dernières semaines. [11:10] Vincent.Sylvester @hg.zetaworlds.com:8002 :J'ai confirmé quelques-uns de ces messages [11:11] Ubit Umarov : eckk le code semble bon... dang [11:12] Andrew Hellershanks : Vincent, ça a l'air bien. Beaucoup de "not a bug", ou de bogues obsolètes, ou ... ? [11:12] Andrew Hellershanks : Ubit, nous devrons peut-être attendre que Bill y jette un œil. [11:13] Vincent.Sylvester @hg.zetaworlds.com:8002 : J'ai fermé ceux liés à ODE car ils ne s'appliquent plus, quelques uns étaient des bugs qui ont été corrigés ou des patches fusionnés et personne ne l'a remarqué, le reste ne peut pas être reproduit ou les choses ne sont pas vraiment réparables ou raisonnables à mettre en œuvre. [11:13] Andrew Hellershanks : ok, merci. [11:13] Vincent.Sylvester @hg.zetaworlds.com:8002 : Il y a quelques vielles issues confirmées qui m'ont surpris, par exemple des scripts qui ne sont pas sauvegardés correctement lorsqu'ils ne sont pas exécutés. [11:14] Vincent.Sylvester @hg.zetaworlds.com:8002 : Je pensais que cela avait été corrigé, mais apparemment non. [11:15] Vincent.Sylvester @hg.zetaworlds.com:8002 : J'ai jeté un coup d'oeil à une autre chose après coup, osGetLinkParams qui avait un code un peu négligé... enfin du code que j'écrirais donc forcément mauvais, je teste actuellement une amélioration de ce code. [11:16] Vincent.Sylvester @hg.zetaworlds.com:8002 : Quelque chose pour réduire le nombre d'accès au tableau et le rendre un tout petit peu plus rapide.
Prim 100% transparentes dans les viewers
[11:16] Gavin.Hird @grid.xmir.org:8002 : en parlant d'ODE et de physique, il y a un changement dans la prochaine version de mon viewer où les éléments définis à 100% de transparence ne seront pas rendus, ce qui devrait rendre moins coûteux l'utilisation de prims transparents comme physique. [11:17] Vincent.Sylvester @hg.zetaworlds.com:8002 : Vous l'avez déjà mentionné. Je pensais que tous les viewers le faisaient déjà. [11:17] Andrew Hellershanks : C'est surprenant que ce ne soit pas déjà une fonctionnalité des viewers actuels, Gavin. [11:17] Gavin.Hird @grid.xmir.org:8002 : ils en font le rendu entièrement [11:17] Vincent.Sylvester @hg.zetaworlds.com:8002 : C'est intéressant pour ceux qui ne savent pas comment réimporter leurs maillages avec la physique appropriée. [11:17] Gavin.Hird @grid.xmir.org:8002 : comme dans le passage par le pipeline de retraitement complet. [11:18] Vincent.Sylvester @hg.zetaworlds.com:8002 : Peut-être que pendant que vous y êtes, vous pouvez corriger les shaders d'eau :x [11:19] Andrew Hellershanks : Gavin, est-ce que ça va toujours gérer les cas où seulement une partie de l'objet est réglé à 100% de transparence ? [11:19] Gavin.Hird @grid.xmir.org:8002 : LL fait beaucoup de travail d'optimisation sur le moteur de rendu en ce moment, mais cela ne fonctionne que sous Windows. [11:19] Gavin.Hird @grid.xmir.org:8002 : sur le Mac, il renvoie la moitié des FPS avant les optimisations. [11:19] Gavin.Hird @grid.xmir.org:8002: oui [11:20] Ubit Umarov : si on utilise la transparence principale seulement, non ? [11:20] Andrew Hellershanks : Bien. Je peux imaginer des cas où un côté d'un prim est transparent mais où vous pouvez toujours voir des parties des autres côtés en même temps. [11:20] Ubit Umarov : il... [11:21] Gavin.Hird @grid.xmir.org:8002 : si la face est configurée en trans, elle ne sera pas rendue. [11:21] Ubit Umarov : ouais, une simple option qu'ils auraient pu faire il y a longtemps. [11:22] Gavin.Hird @grid.xmir.org:8002 : bien sûr, mais la caméra ne voit généralement pas à travers l'avant et l'arrière en même temps ? [11:22] Ubit Umarov : dang semble que mon commit de sauvegarde de co2 a un problème. [11:22] Ubit Umarov : le code semble correct... mais je viens de recevoir un autre rapport d'erreur. [11:23] Gavin.Hird @grid.xmir.org:8002 : donc c'est la direction de la vue qui compte pour ce qui est sélectionné pour le rendu. [11:23] Gavin.Hird @grid.xmir.org:8002 : quoi encore ? [11:24] Ubit Umarov : sl ignore l'arrière des prims. [11:24] Ubit Umarov : en cours de rendu
Tests unitaires
[11:24] Vincent.Sylvester @hg.zetaworlds.com:8002 : Ce serait vraiment cool si nous avions plus de tests unitaires vérifiant divers aspects ; en l'état actuel, on attrape pas vraiment beaucoup d'erreurs. [11:24] Andrew Hellershanks : J'ai oublié les tests unitaires dans OS. Je ne les ai pas exécutés depuis des lustres. [11:25] Vincent.Sylvester @hg.zetaworlds.com:8002 : Ils passent tous, mais j'ai eu quelques échecs cataclysmiques dans le passé en faisant des changements mineurs. [11:26] Ubit Umarov : les tests unitaires ne peuvent attraper que très peu de choses. [11:27] Ubit Umarov : il est très difficile de faire de tels tests. [11:27] Ubit Umarov : et ne devrait jamais être fait par le même codeur. [11:28] Andrew Hellershanks: Indeed. I have written about 100 tests for some code I have written and I've barely scratched the surface of what needs to be tested. I've mostly just done testing for some standalone utility subroutines. Other parts of the system are harder to test. [11:28] Ubit Umarov: many of our tests are just bad [11:28] Ubit Umarov: i did remove a few even :P [11:29] Vincent.Sylvester @hg.zetaworlds.com:8002: Realistically probably need ten times what is currently done or even more than that [11:29] Andrew Hellershanks: I wouldn't be surprised by that count. [11:29] Vincent.Sylvester @hg.zetaworlds.com:8002: Project needs another intern to do the honors lol [11:29] Ubit Umarov: some just had wrong assumptiosn about what should be done [11:30] Vincent.Sylvester @hg.zetaworlds.com:8002: The funniest is the versioninfo test, because I put build numbers into it and I keep breaking that each time another digit is added [11:30] Andrew Hellershanks: It shouldn't be too hard to write some tests to check code behaviour for a lot of SOG/SOP operations. [11:31] Vincent.Sylvester @hg.zetaworlds.com:8002: Some presence and teleport operations, few more on linkset stuff, quite a few areas that are really in need [11:32] Vincent.Sylvester @hg.zetaworlds.com:8002: Script engine, but poking around in that is easier said than done [11:33] Andrew Hellershanks: Tests for database operations, including items such as inventory management. [11:34] Andrew Hellershanks: Testing DB operations can get tricky. You have DB table creation and migration for several different database engines. [11:34] Vincent.Sylvester @hg.zetaworlds.com:8002: Inventory still has a bug allowing entry of bad data for parentFolderID which breaks inventory, stumbled upon that ticket as well [11:35] Andrew Hellershanks: Sounds like something a unit test could catch. [11:36] Vincent.Sylvester @hg.zetaworlds.com:8002: Among other things, quite a few of the confirmed tickets I guess translate to a unit test once the underlying issue is resolved [11:36] Andrew Hellershanks: The number of unit tests goes up quickly when you start writing tests to check for normal/expected behaviour then add tests to see what happens when you pass bad or invalid data. [11:37] Vincent.Sylvester @hg.zetaworlds.com:8002: There are a few of these and I'm sure way more that I haven't found yet [11:37] Andrew Hellershanks: For the routines I test in my code I sometimes have as many as half a dozen tests being run against a single function. [11:38] Vincent.Sylvester @hg.zetaworlds.com:8002: 8903 is the ticket id of the inventory thing by the way [11:38] Andrew Hellershanks pulls up the mantis report [11:38] Andrew Hellershanks: ty, Vincent. I'm monitoring that one now so I can look at it more closely later. [11:39] Vincent.Sylvester @hg.zetaworlds.com:8002: It has a patch, huh never changed the status on it, will do that later, but yeah I did add a little check to prevent the worst part about the parent folder getting broken [11:39] Vincent.Sylvester @hg.zetaworlds.com:8002: Rest I guess is down to forcing lowercase on the keys somewhere [11:40] Vincent.Sylvester @hg.zetaworlds.com:8002: Though that doesn't seem to have a negative impact [11:40] Gavin.Hird @grid.xmir.org:8002: Kirsten viewer.- is that still a thing? [11:40] Vincent.Sylvester @hg.zetaworlds.com:8002: Yes apparently there was an update to it this year [11:40] Vincent.Sylvester @hg.zetaworlds.com:8002: I even got it logged in, kinda [11:41] Vincent.Sylvester @hg.zetaworlds.com:8002: After that inventory fiasco I didn't check further yet, though did leave at note for Kirsten to not forget us [11:42] Vincent.Sylvester @hg.zetaworlds.com:8002: Used to work fine with OS in the past I primarily used it before Firestorm because of the pretty rendering [11:42] Andrew Hellershanks: I haven't heard of Kirsten viewer before. [11:42] Gavin.Hird @grid.xmir.org:8002: even if it was updated a year ago, it is probably massively out of date [11:43] Gavin.Hird @grid.xmir.org:8002: it primarily focused on addition rendering options as far as I can remember [11:43] Gavin.Hird @grid.xmir.org:8002: it was a thing up to 2014-ish [11:44] Vincent.Sylvester @hg.zetaworlds.com:8002: I have seen some rather pretty regions hence getting more out of the rendering for some nice pictures seemed worth the effort to me [11:44] Gavin.Hird @grid.xmir.org:8002: not if it distroys your inventory [11:44] Vincent.Sylvester @hg.zetaworlds.com:8002: It did login, but crashed on something if I recall, then it broke my inventory, took me a while to find that [11:45] Vincent.Sylvester @hg.zetaworlds.com:8002: Well the patch on OS end should prevent bad data going in [11:45] Andrew Hellershanks: Should be simple enough to force UUID strings to lower case before it gets sent to the DB code for saving. [11:45] Vincent.Sylvester @hg.zetaworlds.com:8002: It's just the main inventory folder the most parent one, should have nullkey as parent not its own id as parent [11:47] Vincent.Sylvester @hg.zetaworlds.com:8002: Should be a simple first for the viewer too, but I wonder where that came from initially, can you imagine if all SL viewers do this any of them will break inventory [11:47] Vincent.Sylvester @hg.zetaworlds.com:8002: Users trying different viewers isn't unheard of [11:47] Gavin.Hird @grid.xmir.org:8002: all the uuids should be stored with the uuid datatype in the databases [11:47] Gavin.Hird @grid.xmir.org:8002: now it is usually varchar [11:48] Vincent.Sylvester @hg.zetaworlds.com:8002: At least forces 36 chars on all of them now I think [11:48] Gavin.Hird @grid.xmir.org:8002: but of course that is a massive upheaval bopth code side and to migrate existing databases [11:49] Vincent.Sylvester @hg.zetaworlds.com:8002: The case didn't seem to matter much in terms of inventory though it did double up folders because they are not unique, probably need to look into connectors for inventory and force lowercase somewhere in there [11:49] Andrew Hellershanks: If DB queries are done case insensitive there would be no need to migrate existing DBs to deal with the issue. Hosever, there may be other scripts used by a grid that could be affected by the issue. [11:49] Gavin.Hird @grid.xmir.org:8002: postgres db use uuid as type for a number of tables, but not all [11:49] Gavin.Hird @grid.xmir.org:8002: such as assets, but nit inventoryitems [11:50] Vincent.Sylvester @hg.zetaworlds.com:8002: Adding .ToLower() in the right place is trivial, just need to find the right place heh [11:50] Gavin.Hird @grid.xmir.org:8002: uuid datatyep enforces more than just case [11:50] Andrew Hellershanks: Gavin, by "uuid as type" do you mean used as an index? [11:50] Gavin.Hird @grid.xmir.org:8002: it also knows about the 3 different uuid formats that exist [11:51] Gavin.Hird @grid.xmir.org:8002: no it is a native datatype of postgres and modern MySL [11:51] Gavin.Hird @grid.xmir.org:8002: MYSQL [11:51] Ubit Umarov: well that patch has not much relation to the issue [11:51] Ubit Umarov: may fix some strange case,.. [11:52] Ubit Umarov: and has no relation at all with tolower [11:52] Vincent.Sylvester @hg.zetaworlds.com:8002: It just prevents viewers setting parent and folder id to the same [11:52] Vincent.Sylvester @hg.zetaworlds.com:8002: Like I said, it's just that, the rest I haven't looked into yet [11:53] Vincent.Sylvester @hg.zetaworlds.com:8002: tolower needs to go into inventory somewhere... no idea where yet haven't looked [11:53] Ubit Umarov: ( fix the timer issues [11:53] Ubit Umarov: fixed [11:53] Ubit Umarov: can't commit now :) [11:53] Gavin.Hird @grid.xmir.org:8002: of course you can [11:54] Gavin.Hird @grid.xmir.org:8002: just do it ;-) [11:54] Andrew Hellershanks: :) [11:54] Ubit Umarov: bad boy :p [11:54] Vincent.Sylvester @hg.zetaworlds.com:8002: I think I have that patch locally so I'll look into inventory see if I can find the correct place to add the tolower [11:54] Ubit Umarov: wants to see us all killed :) [11:54] Vincent.Sylvester @hg.zetaworlds.com:8002: See what Kirsten viewer breaks next from there lol [11:54] Gavin.Hird @grid.xmir.org:8002: I suggest you don't use it [11:54] Andrew Hellershanks: Ubit, you will just be testing the protection against restarting this place during meeting hours. ;) [11:54] Ubit Umarov: :) [11:55] Gavin.Hird @grid.xmir.org:8002: there is very little point in chasing bugs from unsupported viewers [11:55] Vincent.Sylvester @hg.zetaworlds.com:8002: Wiki think still mentions how you can get SL viewer connected to OpenSim and if Kirsten is based on that others could break things as well [11:55] Andrew Hellershanks: Gavin, That is true if those bugs are only in the one viewer. [11:55] Ubit Umarov: see? one tries to reduce CO2 and gets into worse trouble.. [11:55] Vincent.Sylvester @hg.zetaworlds.com:8002: Not to mention we should protect against bad data no matter from where [11:56] Gavin.Hird @grid.xmir.org:8002: sure, but a better methiod is to convert to using native uuid data type for such data at the backend [11:56] Vincent.Sylvester @hg.zetaworlds.com:8002: I'm looking at my mariadb install and it doesn't seem to have that type yet [11:57] Gavin.Hird @grid.xmir.org:8002: a lot of the code is already in there, only it is only used to the pgsql section of the code [11:58] Gavin.Hird @grid.xmir.org:8002: https://mariadb.com/kb/en/uuid-data-type/ [11:58] Andrew Hellershanks: Ubit, I saw the reference to CO2 in the changelog and meant to ask you what it meant. I don't think it means carbon dioxide. :) [11:59] Selby.Evans @grid.kitely.com:8002: https://www.gofundme.com/f/give-people-free-internet-voice-communication [11:59] Vincent.Sylvester @hg.zetaworlds.com:8002: Ah I am still on 10.6 so that's why [12:00] Selby.Evans @grid.kitely.com:8002: Musr go -- bye all [12:01] Ubit Umarov: thx selby.Evans [12:01] Andrew Hellershanks: ok, Selby. See you next week. [12:01] Andrew Hellershanks: We are at the top of the hour. Any final comments/questions/thoughts for today? [12:02] Jagga Meredith: CO2? [12:02] Andrew Hellershanks: That's what I was also asking. [12:02] Vincent.Sylvester @hg.zetaworlds.com:8002 whispers: It [12:02] Gavin.Hird @grid.xmir.org:8002: who cares about CO2 [12:02] Gavin.Hird @grid.xmir.org:8002: it is atrace gas of 0.04% of the atmopshere [12:03] Vincent.Sylvester @hg.zetaworlds.com:8002: CO2 is what plants crave [12:03] Gavin.Hird @grid.xmir.org:8002: exactly [12:04] Gavin.Hird @grid.xmir.org:8002: uder 280 ppm plant growth goes down dramatically. at 150 ppm it stops completely [12:04] Gavin.Hird @grid.xmir.org:8002: they want us down to around 280 again [12:05] Ubit Umarov: yes co2 [12:05] Ubit Umarov: reducing cpu work saves power, so CO2 :p [12:05] Vincent.Sylvester @hg.zetaworlds.com:8002: Just go kill a cow if the timer thing don't work, CO2 impact likely the same rofl [12:05] Andrew Hellershanks: If that is what it was really about and not something else its time to wrap up todays meeting. [12:06] Ubit Umarov: well less opensim cows may also do it yes [12:06] Gavin.Hird @grid.xmir.org:8002: :-) [12:06] Andrew Hellershanks: Thank you all for coming. See you again next week.