Aye aye, Sir. Eben erledigt, Sir. Soweit ich sehe, keine Probleme mit der Pre3, Sir.ikari_01 hat geschrieben:Kannst du es schonmal mit der pre3 testen? von pre3 auf pre4 habe ich explizit was am MSU1-Timing gedreht.
Chrono Trigger mit Anime-Intro :-)
- Ramsis
- SNES-Freak

- Beiträge: 538
- Registriert: 29. Dezember 2011, 15:33
- +Positive Tradingpoints+: 3 von 3
- Wohnort: Löwenhöhle :3
- Kontaktdaten:
Re: Chrono Trigger mit Anime-Intro :-)
- ikari_01
- snesfreaks.com-Team

- Beiträge: 441
- Registriert: 15. Juni 2010, 23:19
- +Positive Tradingpoints+: 6 von 6
- Wohnort: Wunstorf
Re: Chrono Trigger mit Anime-Intro :-)
Um "Grafikfehler" nochmal zu spezifizieren: Meinst du zwei "grisselige" Scanlines oben und unten am Videobild? Die gehen bei mir weg, wenn ich die ingame-hooks abschalte.
sd2snes news: https://sd2snes.de
- ikari_01
- snesfreaks.com-Team

- Beiträge: 441
- Registriert: 15. Juni 2010, 23:19
- +Positive Tradingpoints+: 6 von 6
- Wohnort: Wunstorf
Re: Chrono Trigger mit Anime-Intro :-)
Hier ist jetzt erstmal Mahlzeit + SHIELD gucken angesagt. 
sd2snes news: https://sd2snes.de
- Ramsis
- SNES-Freak

- Beiträge: 538
- Registriert: 29. Dezember 2011, 15:33
- +Positive Tradingpoints+: 3 von 3
- Wohnort: Löwenhöhle :3
- Kontaktdaten:
Re: Chrono Trigger mit Anime-Intro :-)
Nee, ich meinte größere Blöcke "defekter" Tiles, die (immer nur für wenige Frames) an diversen Stellen im Video auftauchen.ikari_01 hat geschrieben:Um "Grafikfehler" nochmal zu spezifizieren: Meinst du zwei "grisselige" Scanlines oben und unten am Videobild? Die gehen bei mir weg, wenn ich die ingame-hooks abschalte.
Danke für deine Bemühungen, und guten Appetit.
- ikari_01
- snesfreaks.com-Team

- Beiträge: 441
- Registriert: 15. Juni 2010, 23:19
- +Positive Tradingpoints+: 6 von 6
- Wohnort: Wunstorf
Re: Chrono Trigger mit Anime-Intro :-)
Sehen die Tiles denn richtig schrottig aus oder eher wie Tiles, die zum Video gehören, aber an der falschen Stelle sind?
sd2snes news: https://sd2snes.de
- Ramsis
- SNES-Freak

- Beiträge: 538
- Registriert: 29. Dezember 2011, 15:33
- +Positive Tradingpoints+: 3 von 3
- Wohnort: Löwenhöhle :3
- Kontaktdaten:
Re: Chrono Trigger mit Anime-Intro :-)
Hmmm ... um das festzustellen, sind meine Augen wohl zu träge.ikari_01 hat geschrieben:Sehen die Tiles denn richtig schrottig aus oder eher wie Tiles, die zum Video gehören, aber an der falschen Stelle sind?
So, aber jetzt sag ich gute Nacht.
- ikari_01
- snesfreaks.com-Team

- Beiträge: 441
- Registriert: 15. Juni 2010, 23:19
- +Positive Tradingpoints+: 6 von 6
- Wohnort: Wunstorf
Re: Chrono Trigger mit Anime-Intro :-)
Hab die Ursache gefunden und ich bin ausnahmsweise nicht (unmittelbar) schuld 
Es sind borti4938s Änderungen an get_snes_reset_state, die eine Verzögerung von einer Millisekunde beim Auslesen des Reset-Buttons einführen. Dafür ist bei MSU1 nicht genug Zeit und es kommt zu Buffer Underruns. Mal sehen, wie die Kuh vom Eis zu bekommen ist...
Es sind borti4938s Änderungen an get_snes_reset_state, die eine Verzögerung von einer Millisekunde beim Auslesen des Reset-Buttons einführen. Dafür ist bei MSU1 nicht genug Zeit und es kommt zu Buffer Underruns. Mal sehen, wie die Kuh vom Eis zu bekommen ist...
sd2snes news: https://sd2snes.de
- borti4938
- Hardcore SNES-Freak

- Beiträge: 1069
- Registriert: 23. August 2012, 17:43
- +Positive Tradingpoints+: 69 von 69
Re: Chrono Trigger mit Anime-Intro :-)
Whoops
Zu meiner Verteidigung kann ich ja mal sagen, dass die Konzeptionsidee von dir kam 
https://github.com/mrehkopf/sd2snes/pull/44
Nur die Umsetzung ist dann wohl nicht ausgereift, bzw. passt nicht mehr zum neuen Timing. Mein Fehler
Ein Timingproblem mit MSU-1 hatte ich damals mit der Implementierung schon, bis sie so war, wie sie jetzt ist.
Nochmal kurz:
Ziel der Implementierung ist es, einen Doppelreset vom IGR zu detektieren, ohne dass beispielsweise der Controller erneut ausgelesen wird und dadurch ggf. 2x ein einfacher Reset zum Doppelseite interpretiert wird.
Das SNES wird immer wieder im Reset gehalten, während auf ein Freigeben der Resetleitung bzw. auf den potentiellen Doppelreset gewartet wird.
Mein Ansatz wäre erst mal die Wartezeit zu verkürzen:
Die verlinkte Wartezeit (also die Länge) war halt schon in der Implementierung drinnen. Ich meine, die war sogar mal bei 5ms. Man kann aber auch potentiell deutlich kleiner machen. Da - soweit ich weiß, bitte korrigieren, wenn falsch - kein Kondensator in der Resetleitung ist, könnte man auch die Zeit in den einstelligen Mikrosekundenbereich verkürzen. Das sollte mehr als ausreichend sein, damit sich die Leitung auflädt.
Edit: Man kann im Verlinkten Code die Zeilen 107 und 108 in ein if(reset_short_flag){...} einnisten. Dann stört die Verzögerung gar nicht mehr.
Ist die Kuh damit gerettet?
Sorry für die Problemchen
https://github.com/mrehkopf/sd2snes/pull/44
Nur die Umsetzung ist dann wohl nicht ausgereift, bzw. passt nicht mehr zum neuen Timing. Mein Fehler
Ein Timingproblem mit MSU-1 hatte ich damals mit der Implementierung schon, bis sie so war, wie sie jetzt ist.
Nochmal kurz:
Ziel der Implementierung ist es, einen Doppelreset vom IGR zu detektieren, ohne dass beispielsweise der Controller erneut ausgelesen wird und dadurch ggf. 2x ein einfacher Reset zum Doppelseite interpretiert wird.
Das SNES wird immer wieder im Reset gehalten, während auf ein Freigeben der Resetleitung bzw. auf den potentiellen Doppelreset gewartet wird.
Mein Ansatz wäre erst mal die Wartezeit zu verkürzen:
Die verlinkte Wartezeit (also die Länge) war halt schon in der Implementierung drinnen. Ich meine, die war sogar mal bei 5ms. Man kann aber auch potentiell deutlich kleiner machen. Da - soweit ich weiß, bitte korrigieren, wenn falsch - kein Kondensator in der Resetleitung ist, könnte man auch die Zeit in den einstelligen Mikrosekundenbereich verkürzen. Das sollte mehr als ausreichend sein, damit sich die Leitung auflädt.
Edit: Man kann im Verlinkten Code die Zeilen 107 und 108 in ein if(reset_short_flag){...} einnisten. Dann stört die Verzögerung gar nicht mehr.
Sorry für die Problemchen
Zuletzt geändert von borti4938 am 23. April 2015, 08:05, insgesamt 1-mal geändert.
- mupfelofen-de
- SNES-Programmer

- Beiträge: 214
- Registriert: 2. März 2013, 10:38
- +Positive Tradingpoints+: 0 von 0
- Wohnort: Stuttgart
- Kontaktdaten:
Re: Chrono Trigger mit Anime-Intro :-)
Visit my user-profile at shackspace the hackerspace in Stuttgart.
- ikari_01
- snesfreaks.com-Team

- Beiträge: 441
- Registriert: 15. Juni 2010, 23:19
- +Positive Tradingpoints+: 6 von 6
- Wohnort: Wunstorf
Re: Chrono Trigger mit Anime-Intro :-)
Ich schreibe da aber: "As soon as the sd2snes detects the first reset pulse" und nicht "always when polling the reset button state"borti4938 hat geschrieben:WhoopsZu meiner Verteidigung kann ich ja mal sagen, dass die Konzeptionsidee von dir kam
https://github.com/mrehkopf/sd2snes/pull/44
Und mit dem neuen Timing hat es nichts zu tun, ich hab das einfach erst nach der pre3 ins repo gemerged.
sd2snes news: https://sd2snes.de
- borti4938
- Hardcore SNES-Freak

- Beiträge: 1069
- Registriert: 23. August 2012, 17:43
- +Positive Tradingpoints+: 69 von 69
Re: Chrono Trigger mit Anime-Intro :-)
Sag ich ja - mein Flüchtigkeitsfehler
Mit der Abfrage sollte das dann auch behoben sein
Ich dachte, das wäre schon im Mai dabei gewesen, kam aber erst danach.
Ich sollte mal wieder ins Bett gehen für heute
Zumindest war es in meinen Builds seit Juni immer dabei und mit dem Timing hatte ich nie Probleme bis jetzt
Zumindest war es in meinen Builds seit Juni immer dabei und mit dem Timing hatte ich nie Probleme bis jetzt
- Ramsis
- SNES-Freak

- Beiträge: 538
- Registriert: 29. Dezember 2011, 15:33
- +Positive Tradingpoints+: 3 von 3
- Wohnort: Löwenhöhle :3
- Kontaktdaten:
Re: Chrono Trigger mit Anime-Intro :-)
@ikari_01, wow, das ging ja rasant.
Ich hoffe, das Problem lässt sich leicht beheben (bin da bei dir aber zuversichtlich).
Ich wusste gar nicht, dass noch jemand an CT dran ist. Fragt sich natürlich, wie weit dieser DarkShock mit den FMV-Sequenzen bislang gekommen ist. Vielleicht schreib ich ihn mal an. 
Ui, danke für den Link!mupfelofen-de hat geschrieben:Eine Kombi-Version mit dieser hier wäre nett: https://github.com/mlarouche/ChronoTrigger-MSU1
-
Tommy
- SNES-Gamer

- Beiträge: 100
- Registriert: 15. Dezember 2011, 16:14
- +Positive Tradingpoints+: 2 von 2
Re: Chrono Trigger mit Anime-Intro :-)
das wäre ja zusammen mit dem Intro, Zwischensequenzen + deutsch Patch der absolute hammermupfelofen-de hat geschrieben:Eine Kombi-Version mit dieser hier wäre nett: https://github.com/mlarouche/ChronoTrigger-MSU1
- duoGS
- SNES-Programmer

- Beiträge: 108
- Registriert: 9. Dezember 2012, 20:17
- +Positive Tradingpoints+: 4 von 4
Re: Chrono Trigger mit Anime-Intro :-)
Jep, da gebe Ich Tommy recht, das wäre die Krönung für Chrono Trigger. 
Re: Chrono Trigger mit Anime-Intro :-)
Hi Ramsis,
Super patch, ich freue mich auf jede Unterstützung für die msu1 Geschichten (ich bin der der Zelda, SMW und F-Zero gemacht hat).
Es ist wirklich ein wenig schade dass du hier mit DarkShock einen patch für dasselbe game gemacht hast... das ist zuviel des Guten
Registrier dich doch auf zeldix.net
http://www.zeldix.net/t789-chrono-trigger-msu-1
Hier kannst du dich mit DS evtl. absprechen wie ihr das mit den FMV hin bekommt; sonst macht ihr vielleicht wieder die ganze Arbeit parallel und doppelt...
Super patch, ich freue mich auf jede Unterstützung für die msu1 Geschichten (ich bin der der Zelda, SMW und F-Zero gemacht hat).
Es ist wirklich ein wenig schade dass du hier mit DarkShock einen patch für dasselbe game gemacht hast... das ist zuviel des Guten
Registrier dich doch auf zeldix.net
http://www.zeldix.net/t789-chrono-trigger-msu-1
Hier kannst du dich mit DS evtl. absprechen wie ihr das mit den FMV hin bekommt; sonst macht ihr vielleicht wieder die ganze Arbeit parallel und doppelt...
