Assimp - Brainstorming zum Release

[kimmi]

Stimmt, nun habe ich die probleme verstanden. Ich bin drann!

Gruß Kimmi

[Jonathan]

Ich fixe das.

Schon eine Idee, wie man das schön lösen könnte? Nachdem ich dort nochmal in eine Endlosschleife geraten bin, habe ich den rfind-Hack mit -1 wieder eingebaut, aber besonders toll fühle ich mich dabei nicht.

[Jonathan]

Noch ein paar Infos: mit dem rfing-1 hack gibt es zwar keine Endlosschleife mehr, aber dafür bekomme ich eine Exception "Error, T5400: invalid string position". Mein Code sieht folgendermaßen aus:

Code: Alles auswählen

	IOStream* MatFilePtr=m_CurrentIOHandler->Open(MaterialFileName);
	if(NULL==MatFilePtr)
	{
		//try the default mat Library
		if(NULL==MatFilePtr)
		{
			
			MatFilePtr=m_CurrentIOHandler->Open(m_MaterialLibFilename);

(Die Verwendung vom Rückgabewert ist doch so richtig, oder? Also, wenn die Datei nicht gefunden werden konnte, sollte keine Exeption kommen, sondern 0 zurückgegeben werden und mein Code weiterlaufen?)

Dabei ist der Dateiname "C:\Visual Studio 2010\Projects\Apocalyptical Age\OgreTest\Cube.material"

[Aramis]

Schon eine Idee, wie man das schön lösen könnte?

Eher diesen Thread vergessen - mea culpa.

Dafuer hab ich es eben reproduzieren koennen (mit vc 2008).

[Aramis]

Das hier muesste es fixen. (jedenfalls kriege ich damit keine Endlosschleife mehr)

Index: D:/dev/assimp3.svnc/trunk/code/FileSystemFilter.h
===================================================================
--- D:/dev/assimp3.svnc/trunk/code/FileSystemFilter.h (revision 1195)
+++ D:/dev/assimp3.svnc/trunk/code/FileSystemFilter.h (working copy)
@@ -213,12 +213,12 @@
dirsep = in.rfind('\\', last_dirsep);
}

- if (std::string::npos == dirsep) {
+ if (std::string::npos == dirsep || dirsep == 0) {
// we did try this already.
break;
}

- last_dirsep = dirsep;
+ last_dirsep = dirsep-1;

tmp += in.substr(dirsep+1, in.length()-pos);
if (wrapped->Exists(tmp)) {

Im Uebrigen denke ich dass die von dir verlinkte Cpp-Referenz fehlerhaft ist, der Startindex gehoert zur Range dazu.

Edit: Ist mal so eingecheckt.

[Jonathan]

Danke, jetzt geht es :)

Stimmt, nun habe ich die probleme verstanden. Ich bin drann!

Schon Fortschritte?

[kimmi]

Leider bisher nicht. Ich habe mich durch die Änderungen seit dem letzten Relase durchgewühlt und konnte noch nicht feststellen, was falsch läuft.

Gruß Kimmi

[kimmi]

Hi,

irgendwie komme ich dem Fehler nicht auf die Spur, was auch dem Umstand geschuldet ist, dass hier sowohl meine Freundin als auch unsere Tochter mit Erkältungen rum-flaggen. ich habe in den Obj-Änderungen im wesentlichen nur die Line- und Point-Änderungen gesehen. Hat sich da vielleicht irgend etwas an der Post-Processing-Steps geändert? Für mich sieht das ein wenig nach degenerierten Dreiecken oder so etwas aus. Bin über jeden Denkanstoß dankbar!

Gruß Kimmi

[Aramis]

Ich waere dankbar wenn einer von euch die beiden Patches einspielen koennte (ich hocke gerade leider an einer Windows-only Box und das hiesige patch kommt mit den diffs nicht klar):

http://sourceforge.net/tracker/?func=de ... id=1067634

Die Patches sind i.O.

[klickverbot]

Done.

[Aramis]

Danke!

[Jonathan]

Hat sich da vielleicht irgend etwas an der Post-Processing-Steps geändert? Für mich sieht das ein wenig nach degenerierten Dreiecken oder so etwas aus. Bin über jeden Denkanstoß dankbar!

Gut, dann kommt hier einer: Ich bin mir ziemlich sicher, es liegt am PostPro. Ich habe eine Kugel in Blender, wenn ich die Blend-Datei lade, fehlen Dreiecke. Wenn ich die exportiere (in Obj oder Collada) fehlen auch die Dreiecke, außerdem flimmer irgendwelche Dreiecke beim drehen komisch (Assimp-Viewer). Wenn ich die Kugel nach 3ds exportieren, sind die Dreiecke da und es flimmert nichts.
Nun, 3ds trianguliert vielleicht alles, also habe ich die Kugel in Blender trianguliert und siehe da, die Datei lädt richtig. Collada und Obj Import klappt jetzt auch, allerdings flimmert hier immer noch irgendwas. Ich hab mal alle Modelle hochgeladen, *Tris sind jeweils die triangulierten.

Spheres.zip

Kugel in verschiedenen Formaten

(322.5 KiB) 742-mal heruntergeladen

[edit]Mein Grafiker hat mir gerade bestätigt, dass der Zombie auch funktioniert, wenn er vor dem Export trianguliert wird.[/edit]

[Aramis]

Das Flimmern wird vom Viewer verursacht, das ist harmlos.

Vermutlich haengt es mit einer nicht sonderlich alten Aenderung am Triangulator zusammen - der laesst neuerdings auch auf Quads den Ear-Cutting-Algo los nachdem mich freundlicherweise jemand darauf hinwies dass auch Vierecke konkav sein koennen.

[Jonathan]

Das Flimmern wird vom Viewer verursacht, das ist harmlos.

Rein aus Interesse: Woher kommt das? Oder ist das einfach so ein kleiner Bug, bei dem niemand die Ursache kennt, aber er interessiert eigentlich auch nicht weiter?

[Aramis]

Genau das :-) AssimpView sortiert transparente Polys schoen fein saeuberlich, aber irgendwas laeuft dabei hin und wieder mal schief. Ich hab die betreffende Stelle im Quellcode sicher schon 10 Mal durchgeguckt, aber mich dabei immer nur aufs neue drueber geaergert dass eigentlich der ganze Viewer ein sauberes, plattformunabhaengiges Rewrite von Scratch braeuchte :-)

[kimmi]

Ich bau da gerade was mit der ZFXCE2, sofern ich zu so etwas mal kommen würde ...

Gruß Kimmi

[Aramis]

Hab Assimp's boost-workaround mal separat hochgeladen falls jemand mal Interesse daran hat.

https://github.com/acgessler/micro.boost

[eXile]

Ich reg' mich auf, weil Visual Studio mir für make_shared nur 10 Parameter gibt, und ihr kommt mit 7 aus? :) (Und, ja, ich hatte in meinen Code deswegen schon Hacks eingebaut, dass ich via make_pair zwei Parameter zu einem zusammenfasse, um auf die 10-Parameter-Grenze zu kommen.) Oh wie schön wäre die Welt mit variadic templates. Aber eine nette Sammlung habt ihr da.

[kimmi]

Hi Assimp-Team,

welche Bugs und Features wollen wir eigentlich bis zum 3.0 Release gefixed haben? Mir fällt spontan die folgende Liste ein:

• LLVM Compiler Bug
• Obj-Crash ( bin ich drann )
• Fehler im Triangulieren
• ...

Was müsste eurer Meinung nach noch gefixed werden? Lasst uns das mal dingfest machen.

Ich brauche zur Zeit klare Ansagen, was wichtig ist, daher diese Anfrage. Sonst komme ich aufgrund massiven Zeitmangels zu rein gar nichts mehr. Und mit solch einer Liste ist es leichter, irgendwann den Deckel dicht zu machen. Sonst dümpeln wir Weihnachten immer noch in der Pre-Release-Phase rum ;) ( das war nat. klar übertrieben, aber irgendwie hat man ja immer noch was, was in den nächsten Release soll ).

Gruß Kimmi

P.S.: Neuen Fehler aufgenommen.

[Jonathan]

Mit Obj meinst du die Probleme beim Post-Pro triangulieren? Kai fragt mich schon immer, wann das wieder geht :D

Von meiner Seite: Ich vermute, dass im OgreLoader noch irgendwas nicht so ganz sauber läuft, aber ich muss jetzt erstmal Animationen in meinem Spiel implementieren. Sobald der Loader dann quasi produktiv eingesetzt wird, wird sich sicherlich noch das ein oder andere ergeben.

Achja, über folgendes hab ich neulich nochmal nachgedacht: Wir wäre es mit einem schöneren C++ frontend? Zum Beispiel mit Vectoren statt Arrays, und namespaces statt prefixes? Die simple Lösung wäre es, alles ganz normal zu laden, und dann in eine 'moderne' C++ Datenstruktur zu kopieren. Ich meine, durch das ganze post-pro und einiges umherkonvertieren beim Laden der verschiedenen Formate liegt das Design von Assimp eh nicht auf schnellen Ladezeiten. Und alle Daten am Ende einmal zu kopieren sollte nicht wirklich ins Gewicht fallen. Auf der anderen Seite hätte man dann eine schönere Schnittstelle mit der man arbeiten kann.

[Aramis]

800 Downloads in einer Woche und das ohne ein Release. Eventuell bringt Qt5 eine Menge Public Momentum.

Die Watch&Fork Zahlen auf Github sind derweil auch tendentiell nichtlinear steigend. Ich denke wir werden uns auf Dauer tatsaechlich ueberlegen muessen ob wir das Primaer-Repos nicht nach dort umziehen :-) (Wichtigster Punkt dabei: jedesmal wenn ich einen Patch im Git-Format einspielen muss und grade keine Linux-Box vor mir habe werde ich schier wahnsinnig - weshalb es zuletzt auch dankenswerterweise klickverbot erledigt hat, aber es ist keine Dauerloesung.)

PS: Ich mache eine Liste aller kritischen TODOs.

[kimmi]

Von mir aus können wir gerne zu Github überwechseln. Das von Hand-Patchen ist echt Schxxxx!

Gruß Kimmi

[Aramis]

Kann man git-svn eigentlich auch nicht auch umgekehrt benutzen? Also das SVN-Repos aus dem Git-Repos aktualisieren :-)

[kimmi]

So etwas gibt es auf jeden Fall. Keine Ahnung, wie das Tool dann wirklich heißt.

Es wäre hlfreich, wenn du die Showstopper-Bugs aus deiner Sicht heraus mal aufzählst. Dann können wir die vor Release3.0 als zu fixen markieren.

Gruß Kimmi

[Jonathan]

Na ich benutze auch git-svn. Damit kann ich mir Änderungen vom SVN holen und auch zurückschicken. Und es als ganz normales Git-Repository benutzen, also von/zu anderen pullen/pushen. Man braucht dafür kein extra Tool oder so, war alles schon in Tortoise-Git mit drin.

Achja: Heute geht es ja richtig rund:
http://sourceforge.net/projects/assimp/ ... Repository

[Aramis]

Hab im Tracker alle kritischen Fehler auf hoechste Prio gesetzt, siehe auch ML Posting.

[Aramis]

@Jonathan: einige Leute haben via git-svn gepulled, dabei werden jedes Mal alle (neuen) Revisionen gezogen was pro Revision einen Stat-Eintrag generiert :-)

Wahrscheinlich also 2 neue Gitclones an einem Tag.

[Jonathan]

Achso, das macht Sinn. Wobei ich sagen muss, ich hatte kein Bock so lange zu warten und hab dann neulich nur die letzten 20 Versionen oder so gepulled. Absolut alle Revisionen zu laden muss doch ewig dauern :D

[Aramis]

Kleine Ankuendigung in eigener Sache: ich wurde in den diesjaehrigen Google Summer of Code akzeptiert und zwar mit einem doch recht Assimp-nahen Projekt.

http://www.google-melange.com/gsoc/org/ ... 12/blender

Kurzfassung: FBX-Importer fuer Assimp und Assimp in Blender implementieren :-) Damit wird es umso wichtiger, dass wir moeglichst bis Mitte Mai das Release fertig kriegen.

[kimmi]

Gratulation! Ok, weiß ich Bescheid.

Gruß Kimmi