Assimp - Brainstorming zum Release

[Aramis]

Thomas und Ich haben vorhin etwas ueber moegliche Erweiterungen, um endlich Vertexanimationen zu unterstuetzen, gesprochen. Dabei hat sich ein gewisser Konsens, wie auch eine winzig kleine ( :-) ) Meinungsverschiedenheit herauskristallisiert.

Bitte guckt euch die Alternativen mal an:
http://pastebin.com/fpXnEZz5

Allen gemein sind die Aenderungen ganz oben. Mein Vorschlag haelt alle Meshdaten in der Meshstruktur, waehrend die Schrompf-Variante auf diesen zusaetzlichen Level an Indirektion verzichtet und animierte Vertices fuer Meshes direkt im Animationskeyframe unterbringt.

Was meint ihr dazu? Hat jemand einen besseren Vorschlag?

[kimmi]

Zum besseren verständnis:
Worin liegt der Vorteil, animierte Vertices direkt in einem Keyframe unterzubringen? Mesh ist nach meinem verständnis doch der Platz, wo wir allgemein Vertexdaten hinterlegen wollen. Reciht es nicht, im Keyframe Indices auf die Vertices zu halten. Vielleicht könnt ihr jeweils die Vor- und Nachteile eurer Lösungen noch mal beleuchten.

Das macht das Entscheiden leichter, wenn an der Diskussion nicht beigewohnt hat.

Gruß Kimmi

[Schrompf]

Es werden pro Anim-Frame ja nicht einzelne Vertices indiziert oder ausgetauscht, sondern der komplette Satz Positionen, Normalen, Texturkoords eines Meshes.

[kimmi]

Aber wenn man die Vertices auf die Keyframes verteilt, hat man mehr als eine Anlaufstelle für das Auslesen von spezifischen Vertices. Das hat aufgrund der fehlenden Indirektion natürlich Vorteile. Allerdings sind dann Daten halt über die komplette Datenstruktur verteilt -> meiner Meinung nach auch nicht so schön.
Ich habe mir mal einige andere Implementierungen für so einen Fall angesehen und die haben dann zum Beispiel eine eigene Sub-Meshinstanz je Keyframe mit den zugehörigen auszutauschenden Vertice-Componenten vorliegen.
Hm, haben da noch andere Meinungen?

Gruß Kimmi

[Aramis]

Mesh ist nach meinem verständnis doch der Platz, wo wir allgemein Vertexdaten hinterlegen wollen

Sehe ich auch so! Lieber in jedem Mesh einen Satz solcher Patches/Submeshes/Attachments/AnimMeshes (die bislang dafuer verwendeten Namen :-) ) hinterlegen und diese von den, in der Animation definierten, Keyframes aus referenzieren. Hat uebrigens den Vorteil dass all vorwiegend auf Meshes arbeitenden PP-Steps schnell feststellen koennen ob ein Mesh Animationsdaten besitzt, ohne dafuer alle Animationen durchscannen zu muessen.

Die meisten PP-Steps werden naemlich geringe Anpassungen benoetigen. Entweder muss die auf dem Hauptmesh ausgefuehrte Operation auf alle Submeshes uebertragen werden, oder der Step bricht einfach ab und ignoriert dann auch das Hauptmesh. Nachdem Vertexanimationen sowieso eher … antik … sind, wuerde sich oftmals der Mehraufwand die ganzen PP-Steps ‘richtig’ anzupassen nicht lohnen.

[Aramis]

Die auf der ML ‘beschlossenen’ Aenderungen sind eingecheckt.

[Aramis]

… wenn er weiterhin seine privaten Probleme mit GNU Mailman zu unseren macht, fliegt er. Ich hoffe, ich konnte ihm das in meiner Mail auf der Liste eben vermitteln.

[kimmi]

Gut, dass wir die ML als moderiert konfiguriert haben. Ist eingentlich außer Ärger irgendetwas von ihm schon mal gekommen?

Gruß Kimmi
i

[Aramis]

AssimpView hat ein Facelifting erfahren und zeigt nun mehr relevante Informationen an. Mal gucken, vermutlich komme ich in den naechsten Tagen auch mal dazu ein paar der seit vielen Monaten bekannten, kritischen Bugs im Viewer zu fixen.

[Aramis]

Ich denke, es ist Zeit die Sache zu beenden.

Mick, the people who are around here ARE the principal authors. Our spare time is a bit limited, and we need to assign priorities or we won't get anywhere. Actually, my personal priorities do not include helping people who insult me. Your are posting on Assimp's official forums, where the only people who check these forums regularly are its primary contributors. Why do you debate if the people you're talking to are 'hostile', if not 'contemptible' to any serious discussion?

The reason that some of the team members may have felt a bit annoyed (as they've clearly shown) is your tendency to overthrow any existing stuff in order to make Assimp suit YOUR needs. You didn't show much interest in other groups of users, nor do you honour the the core targets of the project AND software design in general (to be honest, I believe that someone who does not know the basics of source control and API stability should not try to teach others the meaning of 'open source spirit'). After all this is not absolutely bad, new ideas from a new perspective can be quite beneficial to any FOSS project. Being the one who handled most of the interaction with you - you can't blame me, I did listen and finally I have been open to the addition of vertex animations.

What's not acceptable, however, is your manner. Your postings are demanding and, along nice words and empty phrases, full of unfriendly attacks on us, who have spent thousands of hours with this project. Your unwillingness to form to conform to conventions (ours, and those of the open source world in general) is slightly ignorant - you can't expect much assistance if you don't play with the rules.

I am not going to take this any further. I am going to fix the remaining issues for this feature to be useful for you, but it will take a bit. Patches to improve vertex animation support are generally appreciated, of course.

Bye, Alex

PS: If any regular reader / team member believes that this reaction is inappropriate and does not represent general consensus, I'd be glad to know.

[kimmi]

Na da bin ich ja mal gespannt, wie er reagiert. ich tippe ja auf massivstes Geflame und schlechte Bewertungen.

Gruß Kimmi

[Chromanoid]

ich finde den Text ziemlich gut und neutral. Viel freundlicher hätte man das eigentlich nicht schreiben können...

edit: lol ich hab seine antwort gerade gelesen. ich würde ihn bannen falls das geht. der hat ja wohl ne vollmeise. ansonsten würde ich ihn offiziell irgnorieren.
Die Bannung lässt sich eigentlich mit seinem post begründen :)

er will sich nicht an coding conventions halten
"By the time you ask anyone to adapt to your conventions you've potentially already lost their interest. "

er will dass du seinen code korrigierst, wenn ein stable release kommen soll :| und sieht nicht ein, dass es schlechter code nicht ins release schaffen sollte :o
"If you want a stable release, branch it, lock it down, and fix everything to your liking, and release it. Anything less is unprofessional and yes inappropriate."
"If you don't like code someone submits then rework it yourself."

Er macht seine probleme, die er mit assimp hat, zu den wichtigsten des ganzen planeten :evil: ich meine er schreibt doch irgendwie permanent irgendwelchen anforderungen, die unbedingt gelöst werden müssen
"No one deserves to be berated (or belittled) for asking that potholes be addressed ever. "

"I'm assuming people here are interested in developing/using a library, not changing the ways one another think. So more of that please if you don't mind."
omg

[kimmi]

So Vögel gab es immer und wird es wohl auch immer geben :). Vielleicht sind wir ihn jetzt ja los.

Gruß Kimmi

[Aramis]

Danke fuer die Drittmeinung, Chromanoid … wenigstens steht die Frage, ob wir einfach zu wenig kooperativ sind, damit nicht mehr im Raum. Bannen geht leider nicht, fuerchte ich … die SF.net–Infrastruktur *scheint* den Fall nicht vorzusehen.

Jetzt sagt er dass er einen Importer fertig hat und einen Patch abliefern will … mal gucken.

PS: http://www.codinghorror.com/blog/2008/0 ... pples.html

[Chromanoid]

na kein problem, meine solidarität habt ihr.

"In my mind open-source software development in purist terms means apply the ethic of Wikipedia more or less to software. If you have distaste for people bottle it up, never let it out, just deal with the technical issues at hand. If people are getting all personal let them know it's not appropriate. If you don't like code someone submits then rework it yourself. If you want a stable release, branch it, lock it down, and fix everything to your liking, and release it. Anything less is unprofessional and yes inappropriate. Whether or not any culture like that actually exists anywhere is beside the point. It's an understandable POV I think. By the time you ask anyone to adapt to your conventions you've potentially already lost their interest."

sein wikipedia vergleich hinkt übrigens ziemlich :) die haben da eigentlich eine ziemlich starke qualitätskontrolle, haben ständig tierische trollkriege und so weiter... man kann ihm eigentlich nur antworten: "wenn man sich nicht an konventionen anpassen will, dann hat man in der welt der menschen nichts zu suchen..." denn ohne konventionen würden wir wohl noch auf den bäumen leben...

edit: sorry falls es euch stört, aber ich konnte mir eine antwort auf seinen tread nicht verkneifen :). wenn das jetzt weiter ausarten sollte, verspreche ich, dass ich ihn beschäftige und ihr nicht mit ihm schreiben müsst :D

[kimmi]

Ich denke, daß jede Antwort, die wir nicht tippen müssen und die ihn mehr auf Abstand hält eine gute Antwort ist :).

Gruß Kimmi

[Aramis]

Wenn ihr einverstanden seid werde ich den Patch, den er gepostet hat, bei Gelegenheit mal durchgucken und einchecken, sofern mir daran nichts uebles auffaellt. Was ich nicht einsehe ist allerdings, dass ich den existierenden MDL Code aendern soll – er soll seinen Importer weiter vorne registrieren und sicherstellen dass dieser Dateien, die nicht zu ihm gehoeren, zurueckweist.

[kimmi]

Jupp, dem stimme ich zu. Ich hatte seine Äußerung bezüglich der Importer-Registrierung zunächst voll falsch verstanden. Glücklicherweise habt ihr da noch mal genauer drauf geschaut. Ich bin gespannt, was er an Code einreichen wird.

Gruß Kimmi

[klickverbot]

Ich bin ja mal gespannt, wie sich die Situation noch entwickeln wird. Nach seinen letzten Kommentaren war ich schon fast wieder soweit, mich in die Diskussion einzumischen.

Meiner Meinung nach ist die jetzige Strategie eh das einzig Richtige, das wir machen können, nämlich versuchen, ihn ruhig und freundlich, aber trotzdem bestimmt auf den Boden der Tatsachen zurückzuholen. Auch ich kann übrigens aus meiner Open-Source-Erfahrung bestätigen, dass er eine völlig weltfremde Einstellung hat…

Edit: Ein sehenswerter Vortrag: http://video.google.com/videoplay?docid ... 1522818645

[Aramis]

Ochja, ich hab deinen Edit erst jetzt gesehen. Wirklich sehenswert. Ich sehe mickp die ganze Zeit vor mir :-)

Uebrigens hab ich gesehen dass ein neuer Loader hinzugekommen ist -- was hat der fuer einen Status? Ich will gerne mal testen, konnte aber keine Testfiles finden :-)

Zum Blender-Loader waere noch zu erwaehnen dass er Fortschritte macht. Aktuell werden die meisten statischen Blender-Szenen geladen, (meistens) auch mit Materialien.

[kimmi]

Zur Zeit wird nichts in Assimp-Datestrukturen geladen, deswegen ist da noch kein Testfile zu finden. Das ist der Quake-3-Loader. Dafür mußte ich noch unzip in die Contribs einchecken, da die PK3-Files nicht anderes sind als Zip-Archive. Wenn man sich etwas ansehen kann, schreie ich!

Gruß Kimmi

[Aramis]

Geht klar. Ich hab die vc8 Workspaces nun auch aktualisiert, nachdem Thomas ja schon bei vc9 aktiv wurde. Der tuple-Bug sollte auch gefixt sein -- sorry, ich hab etwas zu spaet realisiert dass es um den Workaround ging und dass ich das verbockt hatte.

[kimmi]

Wegen Tuple-Bug: kein problem. Ich habe es bisher noch nicht geschafft, mein Linux anzuschmeißen, sonst hätte ich mich chon darum gekümmert.

Gruß Kimmi

[Aramis]

Hab uns mal zum Collada-Wiki hinzugefuegt:

https://collada.org/mediawiki/index.php ... _directory
https://collada.org/mediawiki/index.php ... rt_Library

Wie sieht es eurer Meinung nach mit Wikipedia aus? Ich bin da eher zoegerlich, denn ich hab einige Seiten von kleinen Libs gesehen, die irgendein uebereifriger Admin als 'vermutlich vom Autor selber erstellt, bla, bla ...' gebrandmarkt hat.

[kimmi]

Und? Wenn das Interesse groß genug ist, werden die in der regel auch wieder zurück eingetragen :), habe ich festgestellt.

Gruß Kimmi

[klickverbot]

Ich habe mal zwei kleine CMake-Fixes committed, mit denen Assimp auch hier auf OS X 10.6 einwandfrei gebaut und installiert wird.

Ich habe übrigens gerade begonnen, an einem kleinen Viewer in D (mit Qt und OpenGL) zu arbeiten, um die SWIG-Bindings für Assimp bzw. mein D-Modul für SWIG zu testen und auch für D so etwas wie Sample Code bereitstellen zu können. Irgendwelche Wünsche/Vorschläge/Anregungen?

[Aramis]

Anregungen hoechstens dass ich auf das Ergebnis gespannt bin :-) Evtl. koennte der Viewer auch der Standardviewer fuer Nicht-Windowse werden, oder AssimpView eines Tages sogar ersetzen (wobei selbiger zwar eine unbrauchbare Quellcodebasis hat, aber eben doch ueber eine ganze Menge Features verfuegt …).

Uebrigens hat auch Matthias im C#-Binding etwas Viewer-Artiges am Laufen.

[kimmi]

Man kann übrigens mittlerweile auch mit der ZFXCE Modelle von Assimp geladen ansehen. Das Ganze wird mit OpenGL gerendert.

Gruß Kimmi

[Aramis]

Dank Chromanoid haben wir nun auch ein neues Mini-Icon, das auch als Threadicon zur Verfuegung steht.

[klickverbot]

Nachdem wieder einmal jemand auf der Mailing-Liste sinngemäß gemeint hat »Ihr solltet CMake nicht benutzen, weil ich es nicht mag« – es hat sich übrigens herausgestellt, dass er eine non-standard-Distribution von MinGW benutzt, also ganze zwei Pfade in der GUI händisch angeben muss:

Ich denke, wir sollten uns eine offizielle Position darüber zurechtlegen, welche Build-System wir unterstützen bzw. anbieten. Mein Vorschlag wäre, die Makefiles, XCode-Workspaces, etc. im Repository zu lassen, sofern sie nützlich für jemanden sind, aber ausdrücklich darauf hinzuweisen, dass sie nicht unterstützt werden, und zwar funktionieren können, das aber nicht müssen. CMake würde ich dann als Standard-Build-System deklarieren, das auf allen Plattformen (ich selbst habe zwei Linux-Varianten, Windows MSVC/MinGW und Mac OS X GCC 4.2 getestet) zumindest irgendein funktionierendes Ergebnis produziert.

Was haltet ihr davon?

Ach ja, ich habe dazu absichtlich (noch) kein Thema auf der Mailing-List erstellt, weil das Kern-Team ja eh hier mitliest und ich nervige Zwischenrufe gerne vermeiden würde, bis sich wenigstens die Committer einig sind…